If defaultImagePath is only used here, then might be better to just construct the string here instead of declare and set the defaultImagePath variable above.
Cherry picked in the DEN-16839-UI-BN-Whitney-1B1. b53f9e9ad7304e8be3c516517ed1d06d775633b1 The follow up CR would be http://devapps.diality.us:8060/cru/#CLOUDSYNC-DEN-16839-1CFR-57808 Will be closed.
Is there a recirculation mode when there is no water generation (per system team state machine)? or recirculation handled part of sub state of Gen water mode (I guess the name of mode (GENW) sounds always generating water, better to have recirculation as a mode instead of sub state?)
I have introduced two additional build flags in 'DDCommon.h' (ENABLE_ALARM_1 and ENABLE_ALARM_2) to currently stop triggering alarms due to hardware support etc., so. it should be safe for now. if still any specific alarms triggers, will try to disable those alarms.
The intention of this to override was to exercise the control loop state machine since this isnt the variable used to set the pwm, but one layer above. We already have the set pwm ( and dialin functionality ) in the fluid pump driver. Lets discuss in the in person meeting.
