dgfirmware / firmware / App

Modes

Clone Tools

Stats

Commits this week: 0
Total Committers: 15
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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 29 Apr 2021
1:01 pm

Sean Nash

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

Options
    • -0
    • +5
    ./ModeFault.c
    • -0
    • +14
    ./OperationModes.c
    • -0
    • +1
    ./OperationModes.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
    • -36
    • +33
    ./ModeFill.c
  2. … 2 more files in changeset.
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
    • -0
    • +2
    ./ModeRecirculate.c
  2. … 3 more files in changeset.
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.

Tuesday 27 Apr 2021
10:01 am

Dara Navaei

Monday 26 Apr 2021
5:43 pm

Dara Navaei

DEN-7568 added reservoir check to heat disinfect

Options
    • -49
    • +43
    ./ModeHeatDisinfect.c
4:33 pm

Dara Navaei

DEN-7568 tested mode flush

Options
    • -3
    • +7
    ./ModeFlush.c
    • -0
    • +3
    ./ModeHeatDisinfect.c
  3. … 3 more files in changeset.
3:19 pm

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Change flush line RO pump rate to 0.6

Options
    • -1
    • +1
    ./ModeRecirculate.c
11:05 am

qnguyen  (deleted user)

DG-DEN-7802_DG-DEV Dialysate Generation
DG-DEN-7802_DG-DEV Dialysate Generation
10:33 am

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:32 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:17 am

Dara Navaei

DEN-7568 addressed code review comments

Options
    • -18
    • +22
    ./ModeFlush.c
  2. … 3 more files in changeset.
10:16 am

Dara Navaei

Done.

Done.

Sunday 25 Apr 2021
10:36 pm

qnguyen  (deleted user)

This struct needs doxygen comment on individual member.

This struct needs doxygen comment on individual member.

9:00 am

Dara Navaei

DEN-7568 added more states to flush mode

Options
    • -81
    • +134
    ./ModeFlush.c
  2. … 2 more files in changeset.
Saturday 24 Apr 2021
11:04 am

Dara Navaei

Moved this function to the FaultMode.c.

Moved this function to the FaultMode.c.

11:04 am

Dara Navaei

DEN-7568 added more states to DG flush mode

Options
    • -1
    • +40
    ./ModeFault.c
    • -0
    • +2
    ./ModeFault.h
    • -80
    • +121
    ./ModeFlush.c
    • -0
    • +1
    ./ModeFlush.h
    • -1
    • +0
    ./ModeRecirculate.c
  6. … 1 more file in changeset.
Friday 23 Apr 2021
11:41 am

Sean Nash

DEN-7605: Removed heater start from recirculate mode.

Options
    • -1
    • +0
    ./ModeRecirculate.c
10:36 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:35 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:35 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

10:34 am

qnguyen  (deleted user)

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.