Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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?

Should this be an enumeration somewhere?

Should this be an enumeration somewhere?

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
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
yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( ...

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( conductivity sensors ).

Fixed

Fixed

added Todo comment

added Todo comment

They were added while debugging fp_test_configs and why it's never being updated. Removed.

They were added while debugging fp_test_configs and why it's never being updated.
Removed.

tip: use the logger and set the statement to error level and you can see it in runtime

tip: use the logger and set the statement to error level and you can see it in runtime