Quang changed the way persistent alarms work (from task interval counts to ms). So task interval is no longer needed to determine when an alarm condition has persisted for too long.
Please revert this back. It's not always an integer and as you may noticed value is the alias for the _progressRect.value and if you go inside the ProgressRect the value is real and not an integer. Please think about the rest of the code when changing a basic code. if you need to have an integer value in your end code format the value in your specific situation, Also the Slider component has a decimal attribute inherited from RangeRect, see if it's useful in you situation. Thanks,
It's a very immediate value setting to sending value to the view. Also sending these values before checking the dependencies to the other values seems redundant. I suggest keeping the values and on confirm (or whatever the confirm button is) sending the final values to the view. Please apply this for performance.
Can we fulfill this TODO now with the changes to the RO pump? Suggest replace with setROPumpTargetFlowRate( FILL_TARGET_RO_FLOW_RATE, FILL_TARGET_RO_PRESSURE_PSI ).
