Controllers

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

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

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

  1. … 7 more files in changeset.
DEN-7605: Added load cell velocityto load cell monitor.

  1. … 1 more file in changeset.
DEN-7605: Added FPGA reported comm error to FPGA comm check. Added check of load cell ADC errors.

  1. … 2 more files in changeset.
DEN-7802: Added fpga fault check for concentrate pumps

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

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

  1. … 5 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 s/w faults for voltage monitor.

  1. … 1 more file in changeset.
DEN-7605: Added transfer functions for monitored voltages.

  1. … 2 more files in changeset.
DEN-7802: Removed alarm data type and unused variable

  1. … 1 more file in changeset.
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. … 1 more file 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. … 2 more files in changeset.
DEN-8030 debugged and fixed UV reactors driver

  1. … 8 more files in changeset.
DEN-7802: Addressed code review

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.

DG-DEN-7802_DG-DEV Dialysate Generation
DG-DEN-7802_DG-DEV Dialysate Generation