TIA Portal v18: Adding a rung - should be easy right?

[Mas01]

... which will NEVER work the way it is coded with the set/reset.

And when you talk about ladder, say the outputs either assign ("set") the value of a bit (discrete tag) to 1 (not TRUE), or assign ("reset") the value of a bit to 0 (not FALSE); rungs states are either TRUE or FALSE. It may seem to be a semantic point, I know, but trust me, you will thank me later when you are the hero who diagnoses a subtle problem someday cf. here.

Uncle drbitboy

Cheers for this, drb. I liked the video and have subscribed to the channel. Good clear explanations.

 

[drbitboy]

The Right Way.

​

 

[Mas01]

The Right Way.

View attachment 66136​

This is brilliant. Very good of you to help me out like this, cheers.

 

[parky]

To be honest, there is no need for the latch (S/R) the code you posted assumes that the enable (visibility) of the button bit will be true until one or both those or all three of those bits driving the set are true until they go false just use thos e bits onto an out instruction I assume you reset the enable bit or the gate sensor goes false after the button is pressed & the data is stored.
you need to get your head round the flow of the logic it is a common mistake learners make by set/reset loads of bits in a program where they are not required. It makes the program messy, & difficult to find problems.
It seems obvious that the button only needs to be enabled while those bits are on, as your reset is essentially just the not contacts then there is no need for the set/reset as the button enable bit will be true while the logic driving it is true.
As UI have said before either think about the logic & a better way is to use a variable as a sequence variable. i.e. rather than set/reset bits as the sequence use the var, benefits are one variable that tells you the step (i.e. step 1, Step 2.... & so on). It is also useful for driving text lists for the HMI so at a particular step number it displays a text message for the operator. for example:
Step 10 (Var = 10) displays "Press Save Button to Save the value"

 

[Mas01]

And here is the proof.

View attachment 66135​

Excellent proof!
You deserve a beer!...

 

[Mas01]

To be honest, there is no need for the latch (S/R) the code you posted assumes that the enable (visibility) of the button bit will be true until one or both those or all three of those bits driving the set are true until they go false just use thos e bits onto an out instruction I assume you reset the enable bit or the gate sensor goes false after the button is pressed & the data is stored.
you need to get your head round the flow of the logic it is a common mistake learners make by set/reset loads of bits in a program where they are not required. It makes the program messy, & difficult to find problems.
It seems obvious that the button only needs to be enabled while those bits are on, as your reset is essentially just the not contacts then there is no need for the set/reset as the button enable bit will be true while the logic driving it is true.
As UI have said before either think about the logic & a better way is to use a variable as a sequence variable. i.e. rather than set/reset bits as the sequence use the var, benefits are one variable that tells you the step (i.e. step 1, Step 2.... & so on). It is also useful for driving text lists for the HMI so at a particular step number it displays a text message for the operator. for example:
Step 10 (Var = 10) displays "Press Save Button to Save the value"

Brilliant - another beer incoming...

 

[parky]

I don't suggest you change your steps on this one as you have already spent time on it but perhaps next time think about using a seq. var rather than setting & reseting bits. there will be times when you need them though