The current implementation and OR and AND logic is a problem , We need a better logic here. for the time being OR is better so that it won't suck up till pressure drops to 4PSI

removed

updated

updated

updated

removed

Remove Sequence step states if not being used.

Update comments ( remove start/end states)

Update TD to DD.

Update alarm properties (DD fault, Priority eti,.)

Remove the empty line.

Or User cancels drain request: To cancel the ongoing drain process. Could you please add the condition as well?

Remove commented code.

Remove D34 valve close.

Add a TODO comment that D83 will be updated to new valve once new valve driver added.

void inside paranthesis.

You can remove the above condition and do the rest of code, except D65 valve opening. Initially this state was waiting for balancing chamber fill to be completed and open D65 valve later. Now that this function will check the condition and skip the valves actions if not met. this is not the expected behavior.

It's better to keep AND condition that d66 pressure must come down to decay pressure before we start vent process.

Open D80, not D85.

Remove the commented code.

Update description across the file.

Update description.

update the description.

You need to initialize the valve states and timer etc., as similar to when you jump from idle state to fill water state. Fill water state just checks the pressure check condition.

Add TODO comment that it should be removed later.

Add comment.

Set default publish interval to 1 sec.

Remove empty line.

Remove empty line.

Remove empty line.