hdfirmware

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 seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 27 Jul 2021
3:36 pm

Quang Nguyen <qnguyen@diality.com>

DEN-8679: Refactor alarm activate functions

Options
    • -24
    • +4
    /firmware/App/Services/AlarmMgmt.c
11:11 am

qnguyen  (deleted user)

Sent events only for changed settings.

Sent events only for changed settings.

11:04 am

qnguyen  (deleted user)

Set UF volume initially to 0 and removed this variable and related function.

Set UF volume initially to 0 and removed this variable and related function.

11:01 am

qnguyen  (deleted user)

It is part of background task in the battery monitor exec function. The initial value is 0, we will wait here till we get status.

It is part of background task in the battery monitor exec function. The initial value is 0, we will wait here till we get status.

10:54 am

qnguyen  (deleted user)

Done. I prefer generally output description, so refactor is easier. Normally, a refactor will often make these variables in the doxygen comment obsolete (variable name change).

Done. I prefer generally output description, so refactor is easier. Normally, a refactor will often make these variables in the doxygen comment obsolete (variable name change).

10:49 am

qnguyen  (deleted user)

Dara Navaei Please take a look.

Dara Navaei Please take a look.

10:49 am

qnguyen  (deleted user)

This message is requested by UI. UI wants to keep it consistent with other request message, in case there is reject reason in the future.

This message is requested by UI. UI wants to keep it consistent with other request message, in case there is reject reason in the future.

10:48 am

qnguyen  (deleted user)

Removed.

Removed.

10:38 am

qnguyen  (deleted user)

If fault mode calls the treatment log data collect function, the record can be zeroed also.

If fault mode calls the treatment log data collect function, the record can be zeroed also.

10:33 am

qnguyen  (deleted user)

The transition function will call this init function.

The transition function will call this init function.

10:29 am

Quang Nguyen <qnguyen@diality.com>

DEN-8679: Addressed code review

Options
    • -21
    • +87
    /firmware/App/Drivers/Battery.c
    • -1
    • +2
    /firmware/App/Drivers/Battery.h
    • -6
    • +3
    /firmware/App/Modes/ModePostTreat.c
    • -4
    • +21
    /firmware/App/Modes/ModeTreatment.c
    • -16
    • +2
    /firmware/App/Modes/ModeTreatmentParams.c
    • -1
    • +0
    /firmware/App/Modes/ModeTreatmentParams.h
    • -1
    • +1
    /firmware/App/Modes/Rinseback.c
    • -0
    • +4
    /firmware/App/Services/SystemComm.c
    • -2
    • +34
    /firmware/App/Services/SystemCommMessages.c
    • -0
    • +3
    /firmware/App/Services/SystemCommMessages.h
10:27 am

qnguyen  (deleted user)

Done.

Done.

10:25 am

qnguyen  (deleted user)

Fixed.

Fixed.

10:24 am

qnguyen  (deleted user)

Fixed.

Fixed.

10:19 am

qnguyen  (deleted user)

Done.

Done.

10:18 am

qnguyen  (deleted user)

The else is when AC power detected. Added clear alarm condition.

The else is when AC power detected. Added clear alarm condition.

Monday 26 Jul 2021
10:52 pm

Dara Navaei

DEN-9054 changed the build history file form text to log

Options
    • -0
    • +18
    /firmware/hd_build_history.log
    • -9
    • +0
    /firmware/hd_build_history.txt
5:49 pm

Dara Navaei

DEN-9054 aligned the build history file

Options
    • -8
    • +7
    /firmware/hd_build_history.txt
5:12 pm

Sean Nash

DEN-8886: Added UI compatibility POST test. Updated alarm audio per latest HDD. Added handling of UI and DG version messages. Fixed watchdog POST. Added recovery check to watchdog POST.

Options
    • -0
    • +25
    /firmware/App/HDCommon.h
    • -4
    • +0
    /firmware/App/Modes/BloodPrime.h
    • -12
    • +43
    /firmware/App/Modes/ModeInitPOST.c
    • -0
    • +2
    /firmware/App/Modes/ModeInitPOST.h
    • -4
    • +29
    /firmware/App/Services/AlarmMgmt.c
    • -3
    • +10
    /firmware/App/Services/FPGA.c
    • -1
    • +1
    /firmware/App/Services/FPGA.h
    • -0
    • +4
    /firmware/App/Services/SystemComm.c
    • -6
    • +34
    /firmware/App/Services/SystemCommMessages.c
    • -3
    • +7
    /firmware/App/Services/SystemCommMessages.h
    • -8
    • +28
    /firmware/App/Services/WatchdogMgmt.c
    • -1
    • +1
    /firmware/HD.dil
4:38 pm

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/HDCommon.h
4:34 pm

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/HDCommon.h
3:36 pm

Quang Nguyen <qnguyen@diality.com>

Merge branch 'DEN-8886-hd_dg_dev-self-tests-2-of-2' into develop

Options
    • -21
    • +21
    /firmware/App/Controllers/BloodFlow.c
    • -0
    • +2
    /firmware/App/Controllers/Bubble.c
    • -25
    • +37
    /firmware/App/Controllers/DialInFlow.c
    • -39
    • +44
    /firmware/App/Controllers/DialOutFlow.c
    • -7
    • +10
    /firmware/App/HDCommon.h
    • -33
    • +169
    /firmware/App/Modes/BloodPrime.c
    • -1
    • +1
    /firmware/App/Modes/Dialysis.c
    • -13
    • +63
    /firmware/App/Modes/ModeInitPOST.c
    • -0
    • +2
    /firmware/App/Modes/ModePreTreat.c
    • -1
    • +1
    /firmware/App/Modes/ModeTreatmentParams.c
    • -28
    • +68
    /firmware/App/Services/AlarmMgmt.c
    • -3
    • +22
    /firmware/App/Services/FPGA.c
    • -0
    • +2
    /firmware/App/Services/FPGA.h
    • -26
    • +7
    /firmware/App/Services/Interrupts.c
    • -26
    • +12
    /firmware/App/Services/SystemComm.c
  16. … 4 more files in changeset.
3:33 pm

Quang Nguyen <qnguyen@diality.com>

Merge branch 'DEN-7802-dg_dev-dialysate-generation' into develop

Options
    • -5
    • +57
    /firmware/App/Controllers/DGInterface.c
    • -24
    • +26
    /firmware/App/Controllers/DGInterface.h
    • -0
    • +1
    /firmware/App/HDCommon.h
    • -0
    • +4
    /firmware/App/Modes/ModeInitPOST.c
    • -0
    • +4
    /firmware/App/Modes/ModePreTreat.c
    • -0
    • +2
    /firmware/App/Modes/ModeTreatment.c
    • -16
    • +22
    /firmware/App/Modes/Prime.c
    • -0
    • +10
    /firmware/App/Modes/SelfTests.c
3:31 pm

Quang Nguyen <qnguyen@diality.com>

DEN-7802: Add build flag to disable dialysate temperature

Options
    • -6
    • +4
    /firmware/App/Controllers/DGInterface.c
    • -0
    • +1
    /firmware/App/HDCommon.h
3:15 pm

Quang Nguyen <qnguyen@diality.com>

Remove pack for blood prime data

Options
    • -4
    • +0
    /firmware/App/Modes/BloodPrime.h
    • -4
    • +0
    /firmware/App/Modes/ModeInitPOST.c
    • -1
    • +1
    /firmware/App/Services/SystemCommMessages.c
3:02 pm

Quang Nguyen <qnguyen@diality.com>

Merge branch 'DEN-8679-hd_dev-treatment-log' into develop

Options
    • -1
    • +14
    /firmware/App/Controllers/DGInterface.c
    • -13
    • +26
    /firmware/App/Controllers/DGInterface.h
    • -8
    • +8
    /firmware/App/HDCommon.h
    • -0
    • +6
    /firmware/App/Modes/ModeInitPOST.c
    • -7
    • +73
    /firmware/App/Services/SystemCommMessages.c
    • -0
    • +6
    /firmware/App/Services/SystemCommMessages.h
9:53 am

Sean Nash

Seems like the two states given as data are not very meaningful. Consider just setting before/after data to zero for this event.

Seems like the two states given as data are not very meaningful. Consider just setting before/after data to zero for this event.

9:50 am

Sean Nash

Not sure this variable (and related get function) are needed.

Not sure this variable (and related get function) are needed.

9:49 am

Sean Nash

User may not have changed all 4 settings. Should we only send events for those that actually changed?

User may not have changed all 4 settings. Should we only send events for those that actually changed?

9:41 am

Sean Nash

I think this will be the time we initiate a treatment. Is that what we want? Or should we move this to transition function to capture time of treatment start?

I think this will be the time we initiate a treatment. Is that what we want? Or should we move this to transition function to capture time of treatment start?