Comment Search

Are we sure about the reversing of states here? If we had this backward, I would think the Systems team would have had a much bigger complaint.
The description of their issue suggests that the 8 BC valves were correctly commanded to requested state except for the one they separately commanded earlier.
If we had this backward, I would think they would have said that all 8 valves did the wrong thing.

Add doxygen comment to right.

updated.

Added param looks wrong.

Updated

Why is this flag set to FALSE here in start state while the other request flags are set to FALSE in their completed states?

Obviously not.
You were reminding me about backwards compatibility. This variable and class ensures that even with this major refactor to the enum structure, the current codebase remains working.

Change HD to TD if moved to td directory.

Same (translation)

Remove extra blank line.

Added

All strings are placed in config.py

Rename to add *Stack.qml. Update Duration as well.

I think you can do this in 1 line of code using the RANGE macro in Common.h.

can't we use DialsysateConductivity - local variable instead of getTargetCond()?

Rmove extra blank line

changed direct file path to dynamic user perspective and storing general configurations folder path in config.py

Add newAlarmIndicateStop to output

align the indent or remove blank space before =

For TX states does not require the ".value"

Added the separate line for each test cases, using \ getting error

we can make a function to just

def rinsebackrequest_Check():
return rinsebackrequest == 8

if this works better? but well have alot more functions elsewhere. Let us know Behrouz NematiPour but the majority of the test cases do indeed use the lambda

Fixed

add airTrapLowerEventCount to input

Fixed