You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was running some test on this AOI.
Again, I really like the functionality.
However, there seems to be an issue when using the HiHi, Hi, Lo, and LoLo
I had some issues enabling and disabling each one.
I see the following lines (63-67):
HiHiGate.Inp_Enable := (HiHiGate.Inp_Enable);
HiGate.Inp_Enable := (HiGate.Inp_Enable);
LoGate.Inp_Enable := (LoGate.Inp_Enable);
LoLoGate.Inp_Enable := (LoLoGate.Inp_Enable);
FailGate.Inp_Enable := (FailGate.Inp_Enable);
None of these are aliases.
I think they should be based on how other tags are passed to the Op_Gate AOI.
I aliased them to the Cfg.Has{XXX}Limit.
Thoughts?
Also, when disabling Cfg.Has{XXX}Limit (Or {XXXX}Gate.Inp_Enable), the Cfg_{XXXX}OffDelay prevents the .Out from disabling until the timer done .
Not really an issue since I doubt Cfg.Has{XXX}Limit will toggle during operation, but something I noticed.
I ended up changing the Op_Gate AOl (lines 48-51):
FROM
if (Out & (NOT Sts_Enabled OR NOT Inp_Condition) & Timer.DN) then
Out := 0;
Timer.Reset := 1;
end_if;
TO
if (Out & (NOT Sts_Enabled OR (NOT Inp_Condition & Timer.DN))) then
Out := 0;
Timer.Reset := 1;
end_if;
Thoughts?
Also, I don't believe the the high and low end clamp are functioning.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
I was running some test on this AOI.
Again, I really like the functionality.
However, there seems to be an issue when using the HiHi, Hi, Lo, and LoLo
I had some issues enabling and disabling each one.
I see the following lines (63-67):
HiHiGate.Inp_Enable := (HiHiGate.Inp_Enable);
HiGate.Inp_Enable := (HiGate.Inp_Enable);
LoGate.Inp_Enable := (LoGate.Inp_Enable);
LoLoGate.Inp_Enable := (LoLoGate.Inp_Enable);
FailGate.Inp_Enable := (FailGate.Inp_Enable);
None of these are aliases.
I think they should be based on how other tags are passed to the Op_Gate AOI.
I aliased them to the Cfg.Has{XXX}Limit.
Thoughts?
Also, when disabling Cfg.Has{XXX}Limit (Or {XXXX}Gate.Inp_Enable), the Cfg_{XXXX}OffDelay prevents the .Out from disabling until the timer done .
Not really an issue since I doubt Cfg.Has{XXX}Limit will toggle during operation, but something I noticed.
I ended up changing the Op_Gate AOl (lines 48-51):
FROM
if (Out & (NOT Sts_Enabled OR NOT Inp_Condition) & Timer.DN) then
Out := 0;
Timer.Reset := 1;
end_if;
TO
if (Out & (NOT Sts_Enabled OR (NOT Inp_Condition & Timer.DN))) then
Out := 0;
Timer.Reset := 1;
end_if;
Thoughts?
Also, I don't believe the the high and low end clamp are functioning.
Beta Was this translation helpful? Give feedback.
All reactions