dgfirmware / firmware / App

Controllers

Clone Tools

Stats

Commits this week: 0
Total Committers: 16
Last Commit: 19 Nov
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
Tuesday 04 May 2021
11:09 am

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Added fpga fault check for concentrate pumps

Options
    • -1
    • +7
    ./ConcentratePumps.c
  2. … 2 more files in changeset.
Monday 03 May 2021
5:55 pm

Dara Navaei

DEN-8030 worked on chemical disinfect mode

Options
    • -19
    • +19
    ./TemperatureSensors.c
    • -26
    • +34
    ./UVReactors.c
    • -5
    • +14
    ./UVReactors.h
  4. … 10 more files in changeset.
5:43 pm

Quang Nguyen <qnguyen@diality.com>

DEN-8103: Removed debug print and override type"

Options
    • -1
    • +1
    ./DrainPump.c
    • -1
    • +1
    ./Fans.c
    • -7
    • +7
    ./Heaters.c
    • -1
    • +1
    ./ROPump.c
    • -3
    • +3
    ./UVReactors.c
  6. … 5 more files in changeset.
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
    ./Voltages.c
    • -2
    • +0
    ./Voltages.h
  3. … 1 more file in changeset.
3:41 pm

Sean Nash

DEN-7605: Added transfer functions for monitored voltages.

Options
    • -11
    • +17
    ./Voltages.c
    • -0
    • +3
    ./Voltages.h
  3. … 2 more files in changeset.
Thursday 29 Apr 2021
11:39 am

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Removed alarm data type and unused variable

Options
    • -1
    • +0
    ./DrainPump.c
  2. … 1 more file in changeset.
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
    ./Heaters.c
    • -0
    • +2
    ./Heaters.h
  3. … 1 more file 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
    • -4
    • +12
    ./Heaters.c
    • -14
    • +39
    ./TemperatureSensors.c
  3. … 2 more files in changeset.
4:09 pm

Dara Navaei

DEN-8030 debugged and fixed UV reactors driver

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

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Addressed code review

Options
    • -2
    • +2
    ./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.

Tuesday 27 Apr 2021
10:01 am

Dara Navaei

Monday 26 Apr 2021
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
    • -5
    • +12
    ./Heaters.c
    • -14
    • +39
    ./TemperatureSensors.c
    • -24
    • +24
    ./TemperatureSensors.h
  4. … 1 more file 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.