Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
It's present in fp_defs as FPRecoveryOptions

It's present in fp_defs as FPRecoveryOptions

LDT-3984 addressed review comments

What's the point of this line if you overwrite the lvl_sen value in the next 2 lines? I think the second lvl_sen is not needed, the one without the offset.

What's the point of this line if you overwrite the lvl_sen value in the next 2 lines?
I think the second lvl_sen is not needed, the one without the offset.

LAST_DD_LEVL is in FW not in dialin. In dialin, you use the Num_of_levels, so this one is correct here. Example: In this case Num_of_levels is 4 (D6,D63,D98,D46), if in the override adding 4 to the...

LAST_DD_LEVL is in FW not in dialin.
In dialin, you use the Num_of_levels, so this one is correct here.
Example:
In this case Num_of_levels is 4 (D6,D63,D98,D46), if in the override adding 4 to the fp level sensor, which is 0, the payload will provide 4, which will be between the FIRST_FP_LEVL and LAST_FP_LEVL FW value as P25 index is 4.

Why increment the number when you didn't add a new sensor?

Why increment the number when you didn't add a new sensor?

This should be LAST_DD_LEVL not NUM_OF_LEVELS if DD and FP levels are in same enum

This should be LAST_DD_LEVL not NUM_OF_LEVELS if DD and FP levels are in same enum

LEVEL_OFFSET?

LEVEL_OFFSET?

LDT-3984 fix for conductivie level sensors d46, d63,d98

  1. … 3 more files in changeset.
Should this be an enumeration somewhere?

Should this be an enumeration somewhere?

LDT-3984 conductive level sensor dialin implementation

  1. … 2 more files in changeset.
LEAHI-DIALIN-LDT-3984_(DD, SW) D46 reads 0 with and without liquid in chamber
LEAHI-DIALIN-LDT-3984_(DD, SW) D46 reads 0 with and without liquid in chamber
LEAHI-DIALIN-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
LEAHI-DIALIN-LDT-3218_Update IOFP permeate target flow rate to 800 mL/min (and flow control as needed)
Added

Added

Add a TODO comment to change the message once implemented in FW

Add a TODO comment to change the message once implemented in FW

removed duplicate

removed duplicate

ROPump is repeated

ROPump is repeated

RECOVERY_STATES

RECOVERY_STATES

LEAHI-DIALIN-LDT-696_[TC Implement] LVTC-13
LEAHI-DIALIN-LDT-696_[TC Implement] LVTC-13
- Added RO recovery enum - Updated FP Pump state enums

  1. … 4 more files in changeset.
update params

update params

LEAHI-DIALIN-LDT-3629_FP and DD valve_states_enum are one short of the full valve list
LEAHI-DIALIN-LDT-3629_FP and DD valve_states_enum are one short of the full valve list
renamed dict keys

  1. … 2 more files in changeset.