StateFlushPermeate.c

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 2 more files in changeset.
LEAHI-DD-FIRMWARE-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
LEAHI-DD-FIRMWARE-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
LDT-3218 updated RO target flow from 750 to 800

  1. … 2 more files in changeset.
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 3 more files in changeset.
LDT-1925 addressed review comments

  1. … 3 more files in changeset.
Updated

Updated

Added

Added

Updated

Updated

Fixed

Fixed

TRUE if low flow alarm is exempt, FALSE if not.

TRUE if low flow alarm is exempt, FALSE if not.

timeSinceP12Started?

timeSinceP12Started?

Should we have a comment for this one too?

Should we have a comment for this one too?

Should we rename this function to "isROPumpLowflowAlarmExempt"? The current name makes me think it will set the alarm to be exempt.

Should we rename this function to "isROPumpLowflowAlarmExempt"? The current name makes me think it will set the alarm to be exempt.

Updated to use isPersistentAlarmTriggered

Updated to use isPersistentAlarmTriggered

Fixed

Fixed

Updated

Updated

Updated

Updated

Changed the implementation

Changed the implementation

Fixed

Fixed

LDT-1925 restructed the code per SRS behavior

  1. … 3 more files in changeset.
Why are we setting timer for pressure and PWM controls? Isn't it only applicable in flow control? And why aren't we resetting exempt flag in same place as timer?

Why are we setting timer for pressure and PWM controls? Isn't it only applicable in flow control?
And why aren't we resetting exempt flag in same place as timer?

I think this function should return TRUE if exempt (we are in first 10 seconds of flow control from off state). All criteria checked here in this function.

I think this function should return TRUE if exempt (we are in first 10 seconds of flow control from off state). All criteria checked here in this function.

I think timer should be internal (private) to this unit.

I think timer should be internal (private) to this unit.