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
Thursday 24 Feb 2022
7:25 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
7:24 pm

Michael Garthwaite

Merge branch 'DEN-12121-sw_dev-sprint-65-michael-garthwaite' into develop

Options
    • -0
    • +4
    ./Services/SystemComm.c
7:24 pm

Michael Garthwaite

DEN-12121: switch case for HD communication override. DIAL-147

Options
    • -0
    • +4
    ./Services/SystemComm.c
6:31 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
6:26 pm

Michael Garthwaite

Merge branch 'DEN-12121-sw_dev-sprint-65-michael-garthwaite' into develop

Options
    • -4
    • +50
    ./Services/SystemComm.c
    • -1
    • +2
    ./Services/SystemComm.h
    • -0
    • +31
    ./Services/SystemCommMessages.c
    • -0
    • +3
    ./Services/SystemCommMessages.h
6:03 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated deliver dialysate state

Options
    • -0
    • +5
    ./Controllers/ConductivitySensors.h
    • -29
    • +57
    ./Modes/ModeFill.c
3:36 pm

Michael Garthwaite

Does this need to be memset when we can initialize it with 0'd values above? (line 92)

Does this need to be memset when we can initialize it with 0'd values above? (line 92)

3:33 pm

Michael Garthwaite

Do we need an if for an always true condition? does this branch statement need to be uncommented?

Do we need an if for an always true condition? does this branch statement need to be uncommented?

3:25 pm

Michael Garthwaite

Shouldn't the override check be its own get function like other overidden properties? Ex: getUVReactorHealth() ?

Shouldn't the override check be its own get function like other overidden properties? Ex: getUVReactorHealth() ?

3:17 pm

Michael Garthwaite

I think this alarm should be not commented out.

I think this alarm should be not commented out.

11:36 am

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
11:29 am

Dara Navaei

Merge branch 'DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up' into develop

Options
    • -3
    • +4
    ./Controllers/ConcentratePumps.c
    • -6
    • +6
    ./Controllers/ROPump.c
11:28 am

Dara Navaei

DEN-11750 updated the concentrate pumps

Options
    • -3
    • +4
    ./Controllers/ConcentratePumps.c
    • -6
    • +6
    ./Controllers/ROPump.c
    • -2
    • +3
    ./Services/Reservoirs.c
Wednesday 23 Feb 2022
8:54 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
8:28 pm

Dara Navaei

Merge branch 'DEN-11928-dg-conductivity-update-den-tbd' into develop

Options
    • -11
    • +23
    ./Controllers/ConductivitySensors.c
    • -9
    • +4
    ./Controllers/ConductivitySensors.h
    • -6
    • +29
    ./Controllers/TemperatureSensors.c
    • -127
    • +254
    ./Modes/ModeFill.c
8:19 pm

Dara Navaei

DEN-12225, DEN-12221 added code to update the UV reactors health and to calculate the target flow change based on the target drain pump RPM

Options
    • -3
    • +12
    ./Controllers/UVReactors.c
    • -5
    • +6
    ./Services/Reservoirs.c
4:31 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated conductivity

Options
    • -2
    • +2
    ./Controllers/ConductivitySensors.c
    • -11
    • +11
    ./Controllers/TemperatureSensors.c
3:29 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated conductivity test

Options
    • -11
    • +23
    ./Controllers/ConductivitySensors.c
    • -1
    • +1
    ./Modes/ModeFill.c
3:00 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated temperature out of range test

Options
    • -3
    • +6
    ./Controllers/TemperatureSensors.c
    • -1
    • +2
    ./Modes/ModeFill.c
2:08 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated temperature, rejection ratio, CPo with new values

Options
    • -2
    • +2
    ./Controllers/ConductivitySensors.c
    • -6
    • +26
    ./Controllers/TemperatureSensors.c
    • -14
    • +13
    ./Modes/ModeFill.c
12:55 pm

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
12:47 pm

Dara Navaei

DEN-12225 updated the UV reactors

Options
    • -1
    • +2
    ./Controllers/UVReactors.c
10:53 am

Dara Navaei

Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Options
    • -1
    • +1
    ./DGCommon.h
10:52 am

Dara Navaei

Merge branch 'DEN-11750-dg_dev-dg-dialysate-temperature-control-tune-up' into develop

Options
    • -1
    • +1
    ./Controllers/ConcentratePumps.c
    • -1
    • +1
    ./Controllers/Heaters.c
    • -7
    • +11
    ./Controllers/ROPump.c
    • -1
    • +1
    ./DGCommon.h
10:39 am

Dara Navaei

DEN-11750 updated the code

Options
    • -1
    • +1
    ./Controllers/Fans.c
    • -1
    • +1
    ./Controllers/Heaters.c
    • -7
    • +11
    ./Controllers/ROPump.c
    • -1
    • +2
    ./Controllers/ROPump.h
    • -1
    • +1
    ./DGCommon.h
    • -1
    • +2
    ./Services/Reservoirs.c
10:14 am

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Implemented conductivity updates

Options
    • -1
    • +1
    ./Controllers/ConcentratePumps.h
    • -9
    • +4
    ./Controllers/ConductivitySensors.h
    • -116
    • +242
    ./Modes/ModeFill.c
Tuesday 22 Feb 2022
5:16 pm

Michael Garthwaite

Could we move these fields outside of the for loop? They dont need to be initialized NUM_OF_FANS_NAMES times.

Could we move these fields outside of the for loop? They dont need to be initialized NUM_OF_FANS_NAMES times.

5:13 pm

Michael Garthwaite

If the DG is broadcasting these raw values, shouldn't Dialin be sync'd as well to handle these new fields?

If the DG is broadcasting these raw values, shouldn't Dialin be sync'd as well to handle these new fields?

5:07 pm

Michael Garthwaite

All the new timer counters no longer are initialized to 0. I understand that they get set in their respective init()'s to the timer counter, but was this intentional?

All the new timer counters no longer are initialized to 0. I understand that they get set in their respective init()'s to the timer counter, but was this intentional?

5:05 pm

Michael Garthwaite

We should probably keep the = 0;

We should probably keep the = 0;