Modes

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-11543: Unable proceed to the Confirm Treatment when "08-3251-9" Acid Concentrate is selected

The HD FW updated to accept up to 4 acid concentration type index.

Below is the UI DataList.conf for acid concentrate.

[Acid Concentrate]

Fres. Naturalyte

08-1251-1

08-2251-0

08-3251-9

DEN-11098 Incorporated peer review comments

  1. … 1 more file in changeset.
Yes, we could use a single variable as the start time in all the pre-treatment states

Yes, we could use a single variable as the start time in all the pre-treatment states

Replaced "loadcellSteadyVolumeStartTime" with "steadyVolumeSamplingStartTime"

Replaced "loadcellSteadyVolumeStartTime" with "steadyVolumeSamplingStartTime"

Done

Done

Done

Done

No, loadcellSteadyVolumeStartTime was last set when the primeDialysateDialyzerTimeLimit has just expired.

No, loadcellSteadyVolumeStartTime was last set when the primeDialysateDialyzerTimeLimit has just expired.

Done

Done

Done

Done

Done.

Done.

This start time probably needs a new name to better describe what it's doing.

This start time probably needs a new name to better describe what it's doing.

Recommend just setting up pumps and valves in this function rather than calling another function.

Recommend just setting up pumps and valves in this function rather than calling another function.

Add blank line between declarations and code.

Add blank line between declarations and code.

This start time was last set in setup function - quite some time ago. So this will have immediately timed out the first time we get here. I think this still works though.

This start time was last set in setup function - quite some time ago. So this will have immediately timed out the first time we get here. I think this still works though.

If skipping priming, would we even get here?

If skipping priming, would we even get here?

Please put declarations at top of scope (i.e. just below the '{' above).

Please put declarations at top of scope (i.e. just below the '{' above).

DEN-11098 Updated code per peer review comments

Fixed.

Fixed.

Replaced by primeDialysateDialyzerTimeLimit because this variable is dynamic and based on the type of dialyzer.

Replaced by primeDialysateDialyzerTimeLimit because this variable is dynamic and based on the type of dialyzer.

Add blank line between these two lines.

Add blank line between these two lines.

Add blank line between these lines.

Add blank line between these lines.

Do we need separate start time variables? These are not used at same time - consider consolidating into a single variable.

Do we need separate start time variables? These are not used at same time - consider consolidating into a single variable.

Should be 120 mL divided by 300 mL/min times 60 sec/min = 24 seconds.

Should be 120 mL divided by 300 mL/min times 60 sec/min = 24 seconds.

Why is this commented out?

Why is this commented out?

DEN-11098 Updated time out limits

Merge branch 'DEN-10602_UI_DEV_Add_PatientID_PreTreatment' into develop

  1. … 2 more files in changeset.
DEN-11098 Implemented code update due to requirements change

HD-DEN-10602_UI Dev Add Patient ID Pre Treatment
HD-DEN-10602_UI Dev Add Patient ID Pre Treatment
DEN-11114 worked on the reservoirs and heaters control

  1. … 5 more files in changeset.
Finishing up the story for firmware

DEN-10607: Implementation [Screen]

DEN-10609: Implementation [Treatment Log]