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
Friday 31 Mar 2023
12:49 pm

Dara Navaei

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Options
    • -1
    • +1
    ./Controllers/Heaters.c
12:48 pm

Michael Garthwaite

12:30 pm

James Walter Taylor <jtaylor@diality.com>

DG-DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen

Update for code review.

Options
    • -1
    • +1
    ./Services/SystemComm.c
12:29 pm

jtaylor  (deleted user)

Done.

Done.

10:07 am

wbracken  (deleted user)

Space between ((

Space between ((

9:47 am

Dara Navaei

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

Options
    • -1
    • +1
    ./DGCommon.h
9:45 am

Michael Garthwaite

Merge branch 'DEN-14788-DIAL-222-heater-duty-cycles' into develop

Options
    • -22
    • +27
    ./Controllers/Heaters.c
9:44 am

Sean Nash and wbracken  (deleted user)

9:44 am

Michael Garthwaite

DIAL-222: changed set heater duty cycle to consider overriden values on reset.

Options
    • -19
    • +27
    ./Controllers/Heaters.c
9:14 am

Michael Garthwaite

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

9:03 am

Sean Nash

I think VPi is closed during standby mode, so not sure why we want to check inlet conductivity in this mode. I imagine new system would get installed by service - would need some kind of priming do...

I think VPi is closed during standby mode, so not sure why we want to check inlet conductivity in this mode.
I imagine new system would get installed by service - would need some kind of priming done in service mode, so I'm not worried about that.

8:59 am

James Walter Taylor <jtaylor@diality.com>

HD-DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen

Closing code review comments.

Options
    • -9
    • +7
    ./Services/SystemComm.c
8:54 am

Michael Garthwaite

will these alarms trigger with open air? Ex: if a user has a device shipped to them, will they be greeted with conductivity alarms on first power up before any water has ran through the system? If...

will these alarms trigger with open air?

Ex: if a user has a device shipped to them, will they be greeted with conductivity alarms on first power up before any water has ran through the system? If so, how will they be able to fix it?

8:36 am

jtaylor  (deleted user)

Done. Does HD communicating always account for UI traffic as well?

Done. Does HD communicating always account for UI traffic as well?

8:35 am

jtaylor  (deleted user)

Done

Done

8:22 am

jtaylor  (deleted user)

Reverted.

Reverted.

Thursday 30 Mar 2023
9:20 pm

Dara Navaei

DEN-14999 updated the code based on the alarms table changes

Options
    • -1
    • +0
    ./Controllers/Heaters.c
    • -1
    • +1
    ./Modes/ModeChemicalDisinfect.c
    • -1
    • +1
    ./Modes/ModeChemicalDisinfectFlush.c
    • -15
    • +28
    ./Modes/ModeFill.c
    • -1
    • +1
    ./Modes/ModeFlush.c
    • -1
    • +1
    ./Modes/ModeHeatDisinfect.c
    • -1
    • +1
    ./Modes/ModeHeatDisinfectActiveCool.c
    • -3
    • +45
    ./Modes/ModeStandby.c
9:18 pm

Sean Nash

DEN-14999: Small fix to DG WD POST.

Options
    • -0
    • +5
    ./Services/WatchdogMgmt.c
8:18 pm

Dara Navaei

DEN-14999 updated the code based on the alarms table update

Options
    • -2
    • +1
    ./Controllers/ConductivitySensors.c
    • -12
    • +0
    ./Controllers/Heaters.c
    • -2
    • +1
    ./Controllers/Pressures.c
    • -2
    • +1
    ./Controllers/TemperatureSensors.c
    • -1
    • +1
    ./Modes/ModeChemicalDisinfect.c
    • -1
    • +6
    ./Modes/ModeChemicalDisinfectFlush.c
    • -2
    • +6
    ./Modes/ModeFlush.c
    • -2
    • +6
    ./Modes/ModeHeatDisinfect.c
    • -2
    • +6
    ./Modes/ModeHeatDisinfectActiveCool.c
7:05 pm

Dara Navaei

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

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

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

7:04 pm

Dara Navaei

Merged DEN-14999

Options
    • -5
    • +28
    ./Controllers/ConductivitySensors.c
    • -14
    • +110
    ./Modes/ModeFill.c
    • -1
    • +3
    ./Modes/ModeInitPOST.c
    • -1
    • +5
    ./Services/SystemComm.c
    • -9
    • +113
    ./Services/SystemCommMessages.c
    • -1
    • +14
    ./Services/SystemCommMessages.h
6:58 pm

Sean Nash

I think if HD not communicating, just clear the pending ack list - when HD revives, any pending messages in here are likely moot.

I think if HD not communicating, just clear the pending ack list - when HD revives, any pending messages in here are likely moot.

6:57 pm

Sean Nash

I think it's better to end the if not POST body after the retries decrement (so don't use up retry during post, but still resend msg).

I think it's better to end the if not POST body after the retries decrement (so don't use up retry during post, but still resend msg).

6:34 pm

Dara Navaei

DEN-14999 fixed the pressures self test

Options
    • -0
    • +1
    ./Controllers/Pressures.c
5:56 pm

James Walter Taylor <jtaylor@diality.com>

DEN-15016-system-faults-during-treatment-then-returns-to-the-standby-screen

Updates to hold off processing retries until after INIT, and after communication with HD started.

Options
    • -7
    • +20
    ./Services/SystemComm.c
5:11 pm

Sean Nash

Merge branch 'DEN-14999-alarm-table-updates' of ssh://dvm-linux02:7999/dg/dgfirmware into DEN-14999-alarm-table-updates

Options
    • -5
    • +5
    ./Services/SystemCommMessages.c
5:11 pm

Sean Nash

Fixed handling of HD cmd to change DG reservoir valve config.

Options
    • -2
    • +2
    ./Services/SystemCommMessages.c
5:10 pm

Dara Navaei

DEN-14999, DEN-15030 updated the start DG command function

Options
    • -5
    • +5
    ./Services/SystemCommMessages.c
5:06 pm

Sean Nash

DEN-14999: Updated 24V range alarms.

Options
    • -9
    • +17
    ./Controllers/Voltages.c
    • -1
    • +1
    ./Controllers/Voltages.h