AlarmMgmtSWFaults.h

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LEAHI-RO-FIRMWARE-LDT-828_Stop State - FP
LEAHI-RO-FIRMWARE-LDT-828_Stop State - FP
LEAHI-RO-FIRMWARE-LDT-1242_IOFP - Finalize Permeate Tank Level Control
LEAHI-RO-FIRMWARE-LDT-1242_IOFP - Finalize Permeate Tank Level Control
Calling it regardless of hardware

Calling it regardless of hardware

Follow up with Eliza on TODO question.

Follow up with Eliza on TODO question.

Pushed the commit

Pushed the commit

Done

Done

Done

Done

Looks like it's still here.

Looks like it's still here.

Current builds of the FP are not triggering alarms. Alarms and recovery states ( ex: paused states ) will be implemented and covered in https://diality.atlassian.net/browse/LDT-828 once requirement...

Current builds of the FP are not triggering alarms. Alarms and recovery states ( ex: paused states ) will be implemented and covered in https://diality.atlassian.net/browse/LDT-828 once requirement draft is finalized.

If in paused state due to alarm, if alarm is recoverable we would need a way to go back to in progress state when user clears alarm.

If in paused state due to alarm, if alarm is recoverable we would need a way to go back to in progress state when user clears alarm.

Done.

Done.

Yeah, I agree. We can remove it

Yeah, I agree.
We can remove it

Yes, will be in paused state, until we do the PreGen mode again.

Yes, will be in paused state, until we do the PreGen mode again.

If check completed, are we stuck in this state forever?

If check completed, are we stuck in this state forever?

Is this assignment needed? Seems like it would already have to be FALSE to get into the if.

Is this assignment needed? Seems like it would already have to be FALSE to get into the if.

Align comments.

Align comments.

Add a blank line before group end marker.

Add a blank line before group end marker.

Align comments or remove them (not necessary for function prototypes).

Align comments or remove them (not necessary for function prototypes).

Removed.

Removed.

Removed.

Removed.

Removed.

Removed.

Updated, thanks.

Updated, thanks.

Dont think we will need to control the permeate tank in this state

Dont think we will need to control the permeate tank in this state

Please remove unnecessary #include's like ROPump, PermeateTank, Conductivity, etc.

Please remove unnecessary #include's like ROPump, PermeateTank, Conductivity, etc.

Remove commented function header.

Remove commented function header.

Similar to how we have the if statement above, please assign the state to FP_PRE_GENP_FILTER_FLUSH once we complete the pressure check.

Similar to how we have the if statement above, please assign the state to FP_PRE_GENP_FILTER_FLUSH once we complete the pressure check.