-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
eurotherm: handle missing sensor better #8310
eurotherm: handle missing sensor better #8310
Comments
Freddie plans to make this ticket clearer, if you don't understand it, ask him. |
Rework infoThe Lewis test need to simulate just one addr being disconnected not all of them which it is currently doing. This may need writing a new Lewis command that only simulates the disconnection of one addr. |
Changed points to an 8, due to having to accommodate changes to |
As a developer i would like the eurotherm to handle a missing sensor better. As seen recently, the IOC was configured for sensors 1,2,3 but 1 had been removed. This resulted in calls to read sensor 1 timing out, but the queue of such reads caused other reads on 2 and 3 to time out, though occasionally a read from 2 or 3 would get through, so they were intermittently showing the right value but mostly in alarm.
maybe the read chain could be modified so that if the first read fails it does not continue the others in that chain on that iteration. It looks like
may initiate reads via
record(seq, "$(P)READ") ` every second. given timeouts in reads, i feel one approach may be toAcceptance criteria
communication with a missing sensor does not affect other working sensors
Time in Planning
1:00:39 02/05/24
The text was updated successfully, but these errors were encountered: