hdfirmware / 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 30 Mar 2023
5:28 pm

Sean Nash

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

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

Sean Nash

DEN-14999: Fixed sending of DG valve settings command.

Options
    • -0
    • +2
    ./Services/SystemCommMessages.c
4:12 pm

Dara Navaei

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

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

Sean Nash

DEN-14999: updates to pressure POST test and 24V range alarms.

Options
    • -1
    • +1
    ./Controllers/PresOccl.c
    • -11
    • +25
    ./Controllers/Voltages.c
2:22 pm

Sean Nash

12:23 pm

Michael Garthwaite

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

11:33 am

Sean Nash

I don't know. I believe check in msg should have no payload, but it usually does have a u32 of unknown purpose. I have found that the very first check-in after power up has no payload according to ...

I don't know. I believe check in msg should have no payload, but it usually does have a u32 of unknown purpose. I have found that the very first check-in after power up has no payload according to candump and this was causing alarm 46. Until UI sorts this out, I am not going to check payload.

11:28 am

Dara Navaei

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

Options
    • -1
    • +1
    ./HDCommon.h
11:26 am

Darren Cox <darren_m@cox.net>

Merge DEN-14860 branch

Options
    • -10
    • +10
    ./Services/Reservoirs.c
10:18 am

Michael Garthwaite

What is the other flavor of UI check in message. If it is a different size, shouldn't it be its own separate message?

What is the other flavor of UI check in message. If it is a different size, shouldn't it be its own separate message?

9:47 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

9:44 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

9:41 am

Sean Nash

DEN-14999: Removed obsolete/redundant alarms. Addressed code review comment.

Options
    • -8
    • +1
    ./Modes/Rinseback.c
    • -31
    • +2
    ./Modes/TreatmentRecirc.c
    • -1
    • +1
    ./Services/SystemComm.c
9:40 am

Sean Nash

Warning (13 min) removed since no longer alarming at 15 min.

Warning (13 min) removed since no longer alarming at 15 min.

9:38 am

Sean Nash

Fixed.

Fixed.

Wednesday 29 Mar 2023
11:09 pm

Sean Nash

DEN-14999: on blood leak detection, alarm, but then go to init state to prevent re-trigger after alarm cleared.

Options
    • -0
    • +1
    ./Controllers/BloodLeak.c
4:30 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

4:11 pm

James Walter Taylor <jtaylor@diality.com>

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

Increased message ACK timeouts, for both DG and UI.

Options
    • -3
    • +3
    ./Services/SystemComm.c
4:01 pm

Dara Navaei

HD-DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen
HD-DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen
2:29 pm

dcox  (deleted user)

Updated all headers.

Updated all headers.

2:27 pm

Darren Cox <darren_m@cox.net>

DEN-14860 Updated headers per code review comment.

Options
    • -10
    • +10
    ./Services/Reservoirs.c
12:23 pm

wbracken  (deleted user)

Add timeOfLastUICheckin and uiDidCommunicate.

Add timeOfLastUICheckin and uiDidCommunicate.

11:35 am

Dara Navaei

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

Options
    • -1
    • +1
    ./HDCommon.h
11:22 am

Sean Nash

Merge branch 'DEN-14999-alarm-table-updates' into develop

Options
    • -10
    • +1
    ./Services/SystemCommMessages.c
11:20 am

Sean Nash

DEN-14999: Removed minimum UF change. Added 250ms of persistence to HD fluid leak alarm. Increased no ack retries from 8 to 10 because UI needed more time at startup. Accept UI check-in message regardless of payload size because UI is sending 2 different flavors of check-in message.

Options
    • -1
    • +11
    ./Controllers/FluidLeak.c
    • -1
    • +1
    ./Modes/ModeTreatmentParams.h
    • -1
    • +2
    ./Services/SystemComm.c
    • -10
    • +1
    ./Services/SystemCommMessages.c
Tuesday 28 Mar 2023
2:23 pm

wbracken  (deleted user)

Does the new code result in one less TRUE comparison? Can rinsebackTimerCtr ever be greater than RINSEBACK_DONE_WARNING_TIME?

Does the new code result in one less TRUE comparison? Can rinsebackTimerCtr ever be greater than RINSEBACK_DONE_WARNING_TIME?

1:33 pm

Sean Nash

DEN-14999: Do not alarm after rinseback completed due to 15 min t/o.

Options
    • -13
    • +4
    ./Modes/Rinseback.c
1:29 pm

Dara Navaei

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

Options
    • -1
    • +1
    ./HDCommon.h
1:22 pm

Darren Cox <darren_m@cox.net>

Merge DEN-14860

Options
    • -0
    • +14
    ./Modes/SelfTests.c
    • -47
    • +86
    ./Services/Reservoirs.c
    • -1
    • +1
    ./Services/Reservoirs.h
1:14 pm

Sean Nash

Fixed merge conflict from staging.

Options
    • -2
    • +2
    ./Controllers/AirTrap.c
    • -2
    • +2
    ./Controllers/BloodFlow.c
    • -14
    • +7
    ./Controllers/DialInFlow.c
    • -12
    • +77
    ./Controllers/DialOutFlow.c
    • -26
    • +64
    ./Controllers/PresOccl.c