Modes

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-8030 continued working on chemical disinfect and added V3 DG valves configuration

  1. … 4 more files in changeset.
DEN-8103: Cleaned up TODO and added alarms

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

  1. … 8 more files in changeset.
DEN-8030 worked on chemical disinfect mode

  1. … 8 more files in changeset.
DEN-8103: Removed debug print and override type"

  1. … 7 more files in changeset.
DEN-8103: Added request to mode fault when alarm has fault property

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

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DG-DEN-7605_HD DG Self Tests
DG-DEN-7605_HD DG Self Tests
DEN-7605: Added function to get previous op mode.

Fixed.

Fixed.

Done.

Done.

Fixed. Compared to target temperature set by HD.

Fixed. Compared to target temperature set by HD.

DEN-7802: Addressed code reviews

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

RESOLVED in CODE WALKTHROUGH.

Generally, when writing conditions involving a BOOL, try to form condition to use TRUE or FALSE depending on whichever will favor the safer path (e.g. if a bug or stack/array overflow corrupted the...

Generally, when writing conditions involving a BOOL, try to form condition to use TRUE or FALSE depending on whichever will favor the safer path (e.g. if a bug or stack/array overflow corrupted the BOOL to a value like 0x12345678, this condition would still pass).
So in this condition, we are deciding whether dialysate can be delivered to the reservoir. We would like to err on the side of not delivering to the reservoir (safer state). So we should form the condition in a way that conductivity and temperature ranges being good must be TRUE (1) in order to deliver dialysate to the reservoir. Any other value would result in coming out of dialysate delivery state which is the safer thing to do.

Add blank line between declarations and code.

Add blank line between declarations and code.

Replace "good" with "in range". Is this function checking temp range for alarm or for acceptable for dialysate to go to reservoir? If latter, shouldn't the range be target +/- 2°C?

Replace "good" with "in range". Is this function checking temp range for alarm or for acceptable for dialysate to go to reservoir? If latter, shouldn't the range be target +/- 2°C?

Fixed.

Fixed.

Merged DEN-8030 chemical disinfect

  1. … 3 more files in changeset.
The if should be < and the else if should be >= otherwise there is double-dipping in case the value equals.

The if should be < and the else if should be >= otherwise there is double-dipping in case the value equals.

DEN-7568 added reservoir check to heat disinfect

DEN-7568 tested mode flush

  1. … 3 more files in changeset.
DEN-7802: Change flush line RO pump rate to 0.6

DG-DEN-7802_DG-DEV Dialysate Generation
DG-DEN-7802_DG-DEV Dialysate Generation
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-7568 addressed code review comments

  1. … 3 more files in changeset.