Modes

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-13154: Further fix to DEN-14814 re: do not require syringe removal to re-do preload operation.

  1. … 1 more file in changeset.
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-14834 changed one of the pre-treatment target fills from 600 to 750mL Changed the reservoir's upper limit in treatment from 1950 to 1900mL

  1. … 1 more file in changeset.
Fixed.

Fixed.

Fixed.

Fixed.

Done.

Done.

HD-DEN-14457_Reservoir Drains Before Treatment With Pre Treatment Build Flags Off

Address comment

Why alarm if above low temp? I think >42 or > tgt+4 should be two ways to get same non-safety high temp alarm. And I think <33 or < tgt-4 should be two ways to get same non-safety low temp alarm. D...

Why alarm if above low temp? I think >42 or > tgt+4 should be two ways to get same non-safety high temp alarm.
And I think <33 or < tgt-4 should be two ways to get same non-safety low temp alarm.
Did Systems want to separate these? My understanding was not to separate.

Add blank line before return.

Add blank line before return.

Fixed merge conflicts.

  1. … 1 more file in changeset.
DEN-13154: Removed dry self-test timeout alarm. Fixed bug DEN-14884.

  1. … 2 more files in changeset.
DEN-14834 added code to calculate the heating parameters in pre-treatment and send it to DG in the pre-treatment reservoir management

HD-DEN-14834_RO Flow Drops TO Zero And CP Measured Speed 1500 Temporarily During Pre Treatment
HD-DEN-14834_RO Flow Drops TO Zero And CP Measured Speed 1500 Temporarily During Pre Treatment
HD-DEN-14457-2_Reservoir Drains Before Treatment With Pre Treatment Build Flags Off

Checkpoint. Simplifying AIR_PUMP bug fix.

This will hard fault the HD. Please change to getTreatmentParameterU32()

This will hard fault the HD. Please change to getTreatmentParameterU32()

This will hard fault the HD. Please change to getTreatmentParameterU32()

This will hard fault the HD. Please change to getTreatmentParameterU32()

HD-DEN-14689_Messaging Bugs_MSG_ID_HD_POST_FINAL_TEST_RESULT_fix
HD-DEN-14689_Messaging Bugs_MSG_ID_HD_POST_FINAL_TEST_RESULT_fix
DEN-14689: DEN-14634: fixed HD_POST_FINAL_TEST_RESULT being sent to be sent 1 time instead of 40 times over 2s.

Merge branch 'DEN-14689-messaging-bugs' into develop

  1. … 4 more files in changeset.
Merge branch 'staging' into DEN-14689-messaging-bugs

DEN-14689: message naming sync

  1. … 5 more files in changeset.
Fixed merge conflicts.

  1. … 3 more files in changeset.
DEN-13154: Update to handling of insufficient Heparin alarm per DEN-14814 and DEN-14287.

  1. … 2 more files in changeset.
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

I don't think it's necessary - but I'm ok with either location for declaration. RESOLVED in CODE WALKTHROUGH.

I don't think it's necessary - but I'm ok with either location for declaration.
RESOLVED in CODE WALKTHROUGH.

1. Style, and preference to keep loop control at the top, not buried in the state function. 2. Keep stack size for the state function more predictable. 3. I was considering unwrapping the state mac...

1. Style, and preference to keep loop control at the top, not buried in the state function.
2. Keep stack size for the state function more predictable.
3. I was considering unwrapping the state machine, starting with a separate pause state, that held for priming to advance to the proper state

Ultimately the simplest change may be limited to just this ifndef. No practical reason to move the declaration in that case.

So far as I can tell, this file was included in the code review only. The repository doesn't include these changes, on this branch.

So far as I can tell, this file was included in the code review only. The repository doesn't include these changes, on this branch.

HD-DEN-14860_Recirculation Alarm 68 During High Flow Treatments
HD-DEN-14860_Recirculation Alarm 68 During High Flow Treatments