dgfirmware / firmware

App

Clone Tools

Stats

Commits this week: 0
Total Committers: 17
Last Commit: 19 Nov 25
Commits this week: 0

Lines of code count not available

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Wednesday 05 May 2021
6:10 pm

Dara Navaei

DEN-8030 continued working on chemical disinfect and added V3 DG valves configuration

Options
    • -21
    • +54
    ./Controllers/Valves.c
    • -8
    • +14
    ./Controllers/Valves.h
    • -2
    • +3
    ./DGCommon.h
    • -47
    • +512
    ./Modes/ModeChemicalDisinfect.c
    • -0
    • +25
    ./Modes/ModeDrain.c
    • -0
    • +10
    ./Modes/ModeFault.c
    • -0
    • +60
    ./Modes/ModeFlush.c
    • -11
    • +124
    ./Modes/ModeHeatDisinfect.c
    • -0
    • +3
    ./Modes/ModeRecirculate.c
    • -1
    • +21
    ./Modes/ModeStandby.c
    • -0
    • +8
    ./Services/Reservoirs.c
1:58 am

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Cleaned up TODO and added alarms

Options
    • -42
    • +14
    ./Controllers/Heaters.c
    • -34
    • +2
    ./Controllers/ROPump.c
    • -2
    • +0
    ./Controllers/ROPump.h
    • -1
    • +1
    ./Controllers/Valves.c
    • -8
    • +15
    ./Drivers/InternalADC.c
    • -8
    • +4
    ./Modes/ModeInitPOST.c
    • -9
    • +2
    ./Modes/ModeRecirculate.c
    • -1
    • +1
    ./Modes/ModeSolo.c
    • -0
    • +6
    ./Services/AlarmMgmt.h
    • -65
    • +1
    ./Services/Interrupts.c
    • -7
    • +4
    ./Services/SystemComm.c
Tuesday 04 May 2021
4:30 pm

Sean Nash

Fixed merge conflicts.

Options
    • -14
    • +63
    ./Controllers/LoadCell.c
    • -3
    • +3
    ./Controllers/TemperatureSensors.c
    • -0
    • +5
    ./Modes/ModeFault.c
    • -0
    • +3
    ./Services/AlarmMgmt.h
    • -0
    • +8
    ./Services/SystemComm.c
    • -0
    • +92
    ./Services/SystemCommMessages.c
    • -0
    • +10
    ./Services/SystemCommMessages.h
  8. … 2 more files in changeset.
4:09 pm

Sean Nash

DEN-7605: Added load cell velocityto load cell monitor.

Options
    • -4
    • +33
    ./Controllers/LoadCell.c
    • -0
    • +1
    ./Controllers/LoadCell.h
    • -0
    • +1
    ./Services/AlarmMgmt.h
2:21 pm

Sean Nash

DEN-7605: Added FPGA reported comm error to FPGA comm check. Added check of load cell ADC errors.

Options
    • -9
    • +29
    ./Controllers/LoadCell.c
    • -3
    • +3
    ./Controllers/TemperatureSensors.c
    • -17
    • +25
    ./Services/FPGA.c
    • -2
    • +2
    ./Services/FPGA.h
11:09 am

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Added fpga fault check for concentrate pumps

Options
    • -1
    • +7
    ./Controllers/ConcentratePumps.c
    • -0
    • +13
    ./Services/FPGA.c
    • -0
    • +2
    ./Services/FPGA.h
Monday 03 May 2021
5:55 pm

Dara Navaei

DEN-8030 worked on chemical disinfect mode

Options
    • -19
    • +19
    ./Controllers/TemperatureSensors.c
    • -26
    • +34
    ./Controllers/UVReactors.c
    • -5
    • +14
    ./Controllers/UVReactors.h
    • -0
    • +1
    ./DGCommon.h
    • -15
    • +1166
    ./Modes/ModeChemicalDisinfect.c
    • -0
    • +14
    ./Modes/ModeChemicalDisinfect.h
    • -17
    • +43
    ./Modes/ModeHeatDisinfect.c
    • -1
    • +20
    ./Modes/ModeStandby.c
    • -1
    • +3
    ./Modes/ModeStandby.h
    • -11
    • +1
    ./Services/AlarmMgmt.h
    • -0
    • +4
    ./Services/SystemComm.c
    • -4
    • +68
    ./Services/SystemCommMessages.c
    • -0
    • +7
    ./Services/SystemCommMessages.h
5:43 pm

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Removed debug print and override type"

Options
    • -1
    • +1
    ./Controllers/DrainPump.c
    • -1
    • +1
    ./Controllers/Fans.c
    • -7
    • +7
    ./Controllers/Heaters.c
    • -1
    • +1
    ./Controllers/ROPump.c
    • -3
    • +3
    ./Controllers/UVReactors.c
    • -0
    • +1
    ./Modes/ModeFault.c
    • -2
    • +2
    ./Modes/ModeFlush.c
    • -3
    • +3
    ./Modes/ModeHeatDisinfect.c
    • -91
    • +13
    ./Services/AlarmMgmt.c
    • -0
    • +4
    ./Services/SystemCommMessages.c
5:15 pm

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Added request to mode fault when alarm has fault property

Options
    • -1
    • +3
    ./Modes/ModeFault.c
    • -1
    • +9
    ./Services/AlarmMgmt.c
    • -11
    • +0
    ./Services/AlarmMgmt.h
10:32 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:31 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Sunday 02 May 2021
3:24 pm

Sean Nash

DG-DEN-7605_HD DG Self Tests
DG-DEN-7605_HD DG Self Tests
Friday 30 Apr 2021
3:54 pm

Sean Nash

DEN-7605: Added s/w faults for voltage monitor.

Options
    • -28
    • +4
    ./Controllers/Voltages.c
    • -2
    • +0
    ./Controllers/Voltages.h
    • -3
    • +5
    ./Services/AlarmMgmt.h
3:41 pm

Sean Nash

DEN-7605: Added transfer functions for monitored voltages.

Options
    • -11
    • +17
    ./Controllers/Voltages.c
    • -0
    • +3
    ./Controllers/Voltages.h
    • -24
    • +24
    ./Drivers/InternalADC.c
    • -11
    • +0
    ./Services/AlarmMgmt.h
Thursday 29 Apr 2021
1:01 pm

Sean Nash

DEN-7605: Added function to get previous op mode.

Options
    • -0
    • +5
    ./Modes/ModeFault.c
    • -0
    • +14
    ./Modes/OperationModes.c
    • -0
    • +1
    ./Modes/OperationModes.h
11:39 am

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Removed alarm data type and unused variable

Options
    • -1
    • +0
    ./Controllers/DrainPump.c
    • -11
    • +0
    ./Services/AlarmMgmt.h
11:00 am

qnguyen  (deleted user)

Fixed.

Fixed.

10:59 am

qnguyen  (deleted user)

Done.

Done.

10:59 am

qnguyen  (deleted user)

Fixed. Compared to target temperature set by HD.

Fixed. Compared to target temperature set by HD.

10:59 am

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Addressed code reviews

Options
    • -0
    • +13
    ./Controllers/Heaters.c
    • -0
    • +2
    ./Controllers/Heaters.h
    • -36
    • +33
    ./Modes/ModeFill.c
10:31 am

pmontazemi  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

9:43 am

Sean Nash

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.

9:30 am

Sean Nash

Add blank line between declarations and code.

Add blank line between declarations and code.

9:04 am

Sean Nash

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?

Wednesday 28 Apr 2021
8:17 pm

qnguyen  (deleted user)

Fixed.

Fixed.

5:08 pm

Dara Navaei

Merged DEN-8030 chemical disinfect

Options
    • -4
    • +12
    ./Controllers/Heaters.c
    • -14
    • +39
    ./Controllers/TemperatureSensors.c
    • -0
    • +2
    ./Modes/ModeRecirculate.c
    • -11
    • +0
    ./Services/AlarmMgmt.h
4:09 pm

Dara Navaei

DEN-8030 debugged and fixed UV reactors driver

Options
    • -11
    • +11
    ./Controllers/UVReactors.c
  2. … 8 more files in changeset.
11:20 am

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Addressed code review

Options
    • -2
    • +2
    ./Controllers/ConcentratePumps.c
10:47 am

pmontazemi  (deleted user)

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.