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 08 Mar 2022
4:58 pm

Dara Navaei

Merge branch 'DEN-12224-dg_hd_dev-switches-monitor' into develop

Options
    • -2
    • +3
    /firmware/App/Services/SystemComm.c
    • -33
    • +27
    /firmware/App/Services/SystemCommMessages.c
4:48 pm

Dara Navaei

DEN-12224, DEN-12232 updated the fans driver

Options
    • -20
    • +0
    /firmware/App/Controllers/Fans.c
4:34 pm

Michael Garthwaite

Similiar to how the check is done in ModePostTreat.c, I think we should check if the heparin parameters are > 0.0 than NEARLY_ZERO.

Similiar to how the check is done in ModePostTreat.c, I think we should check if the heparin parameters are > 0.0 than NEARLY_ZERO.

4:33 pm

Michael Garthwaite

Out of scope of the changes here but hepRate should be getting TREATMENT_PARAM_HEPARIN_DISPENSE_RATE instead of TREATMENT_PARAM_HEPARIN_BOLUS_VOLUME.

Out of scope of the changes here but hepRate should be getting TREATMENT_PARAM_HEPARIN_DISPENSE_RATE instead of TREATMENT_PARAM_HEPARIN_BOLUS_VOLUME.

4:28 pm

Sean Nash

Should we check pump track switch too?

Should we check pump track switch too?

4:28 pm

Sean Nash

Should we check pump track switch too?

Should we check pump track switch too?

4:26 pm

Sean Nash

Why does door need to be closed for this state?

Why does door need to be closed for this state?

4:24 pm

Sean Nash

Do we want to check pump track switch too?

Do we want to check pump track switch too?

4:23 pm

Sean Nash

I think you have these switch checks in the right pre-treatment sub-modes, but I think each of these sub-modes will have a "stopped" state in its state machine where it will go if an alarm occurs a...

I think you have these switch checks in the right pre-treatment sub-modes, but I think each of these sub-modes will have a "stopped" state in its state machine where it will go if an alarm occurs and in that state the user should be allowed to open the door to possibly address the issue.
So I think we need to figure out how to exempt those alarm states.

4:07 pm

Sean Nash

If I understand what you did here, I think we're good updating from HALCoGen w/o breaking the build but if we change stack size(s) we will need to manually update the #if 1 section accordingly afte...

If I understand what you did here, I think we're good updating from HALCoGen w/o breaking the build but if we change stack size(s) we will need to manually update the #if 1 section accordingly after HALCoGen updates this file.

3:59 pm

Sean Nash

I don't think this is the right place to clear alarm conditions. This area of code is updating the actual detected switch states. But switch states are overridable so we should consider any overrid...

I don't think this is the right place to clear alarm conditions. This area of code is updating the actual detected switch states. But switch states are overridable so we should consider any overrides when clearing alarm conditions.

3:40 pm

Dara Navaei

DEN-12224, DEN-12232 added a command to set the fans RPM alarm start time offset

Options
    • -63
    • +28
    /firmware/App/Controllers/Fans.c
    • -3
    • +2
    /firmware/App/Controllers/Fans.h
    • -2
    • +3
    /firmware/App/Services/SystemComm.c
    • -33
    • +27
    /firmware/App/Services/SystemCommMessages.c
    • -3
    • +4
    /firmware/App/Services/SystemCommMessages.h
2:28 pm

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/HDCommon.h
2:27 pm

Dara Navaei

Merge branch 'DEN-12224-dg_hd_dev-switches-monitor' into develop

Options
    • -3
    • +6
    /firmware/App/Controllers/SyringePump.c
2:12 pm

Sean Nash

DEN-12224: Fixes to removing config flag handling for released build.

Options
    • -3
    • +6
    /firmware/App/Controllers/SyringePump.c
    • -5
    • +0
    /firmware/App/Modes/ModePreTreat.c
    • -15
    • +24
    /firmware/App/Modes/SampleWater.c
9:14 am

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/HDCommon.h
9:14 am

Dara Navaei

Merged DEN-12224 and resolved the merge conflicts

Options
    • -1
    • +9
    /firmware/App/Controllers/BloodLeak.c
    • -7
    • +7
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -45
    • +73
    /firmware/App/Controllers/DialOutFlow.c
    • -29
    • +48
    /firmware/App/Controllers/PresOccl.c
    • -48
    • +77
    /firmware/App/Controllers/SyringePump.c
    • -2
    • +2
    /firmware/App/HDCommon.h
    • -1
    • +6
    /firmware/App/Modes/ConsumableSelfTest.c
    • -14
    • +19
    /firmware/App/Modes/Rinseback.c
    • -24
    • +37
    /firmware/App/Modes/SelfTests.c
    • -11
    • +11
    /firmware/App/Services/Reservoirs.c
    • -2
    • +2
    /firmware/App/Services/SystemComm.c
    • -101
    • +110
    /firmware/App/Services/SystemCommMessages.c
    • -13
    • +21
    /firmware/App/Services/WatchdogMgmt.c
8:36 am

Sean Nash

Merge branch 'DEN-12224-dg_hd_dev-switches-monitor' of ssh://dvm-linux02:7999/hd/hdfirmware into DEN-12224-dg_hd_dev-switches-monitor

Options
    • -45
    • +70
    /firmware/App/Controllers/SyringePump.c
8:35 am

Sean Nash

DEN-12224: Reset syringe pump direction alarm persistence when pump is off. Bug DEN-12333.

Options
    • -0
    • +3
    /firmware/App/Controllers/SyringePump.c
Monday 07 Mar 2022
5:28 pm

Dara Navaei

DEN-12224 added F and the end of the defines the are floating numbers

Options
    • -1
    • +1
    /firmware/App/Controllers/AirTrap.c
    • -26
    • +26
    /firmware/App/Controllers/BloodFlow.c
    • -1
    • +1
    /firmware/App/Controllers/BloodFlow.h
    • -4
    • +4
    /firmware/App/Controllers/DGInterface.c
    • -1
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -21
    • +21
    /firmware/App/Controllers/DialInFlow.c
    • -23
    • +23
    /firmware/App/Controllers/DialOutFlow.c
    • -8
    • +8
    /firmware/App/Controllers/Fans.c
    • -18
    • +18
    /firmware/App/Controllers/PresOccl.c
    • -36
    • +36
    /firmware/App/Controllers/SyringePump.c
    • -9
    • +9
    /firmware/App/Controllers/Temperatures.c
    • -4
    • +4
    /firmware/App/Controllers/Valves.c
    • -3
    • +3
    /firmware/App/Controllers/Voltages.h
    • -1
    • +1
    /firmware/App/Drivers/InternalADC.h
    • -4
    • +4
    /firmware/App/Drivers/SafetyShutdown.c
  16. … 10 more files in changeset.
4:57 pm

Dara Navaei

DEN-12224 added _RELEASE_ to all the software configurations enums

Options
    • -2
    • +4
    /firmware/App/Controllers/PresOccl.c
    • -0
    • +1
    /firmware/App/Controllers/SyringePump.c
    • -0
    • +4
    /firmware/App/Modes/ConsumableSelfTest.c
    • -5
    • +9
    /firmware/App/Modes/Dialysis.c
    • -0
    • +4
    /firmware/App/Modes/ModeInitPOST.c
    • -0
    • +4
    /firmware/App/Modes/ModePostTreat.c
    • -14
    • +34
    /firmware/App/Modes/ModePreTreat.c
    • -2
    • +15
    /firmware/App/Modes/Prime.c
    • -6
    • +10
    /firmware/App/Modes/Rinseback.c
    • -0
    • +2
    /firmware/App/Modes/SampleWater.c
    • -4
    • +16
    /firmware/App/Modes/SelfTests.c
    • -0
    • +2
    /firmware/App/Services/AlarmMgmt.c
    • -0
    • +6
    /firmware/App/Services/SystemCommMessages.c
    • -8
    • +16
    /firmware/App/Services/WatchdogMgmt.c
    • -0
    • +2
    /firmware/App/Tasks/TaskPriority.c
2:53 pm

Dara Navaei

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

Options
    • -1
    • +1
    /firmware/App/HDCommon.h
2:52 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Added SW_CONFIG changes because committed 667feab319b 10Feb 2022 was missing in committed 3d72b777cf1 15Feb 2022

Options
    • -8
    • +14
    /firmware/App/Modes/ConsumableSelfTest.c
2:35 pm

Dara Navaei

Merge branch 'DEN-12224-dg_hd_dev-switches-monitor' of ssh://dvm-linux02:7999/hd/hdfirmware into DEN-12224-dg_hd_dev-switches-monitor

Options
2:35 pm

Dara Navaei

Merge branch 'DEN-12224-dg_hd_dev-switches-monitor' of ssh://dvm-linux02:7999/hd/hdfirmware into DEN-12224-dg_hd_dev-switches-monitor

Options
    • -5
    • +0
    /firmware/App/Modes/ModeStandby.c
    • -17
    • +0
    /firmware/App/Services/AlarmMgmt.c
    • -2
    • +0
    /firmware/App/Services/AlarmMgmt.h
    • -1
    • +0
    /firmware/App/Services/SystemCommMessages.c
2:34 pm

Dara Navaei

DEN-12224 added _RELEASE_ to the software configurtions

Options
    • -0
    • +2
    /firmware/App/Controllers/AirTrap.c
    • -24
    • +52
    /firmware/App/Controllers/BloodFlow.c
    • -0
    • +8
    /firmware/App/Controllers/BloodLeak.c
    • -38
    • +78
    /firmware/App/Controllers/DialInFlow.c
    • -22
    • +50
    /firmware/App/Controllers/DialOutFlow.c
    • -0
    • +19
    /firmware/App/Controllers/PresOccl.c
    • -9
    • +33
    /firmware/App/Controllers/SyringePump.c
    • -0
    • +3
    /firmware/App/Controllers/Temperatures.c
    • -3
    • +7
    /firmware/App/Modes/BloodPrime.c
2:34 pm

Sean Nash

DEN-12224: Removed no new treatment alarm property.

Options
    • -5
    • +0
    /firmware/App/Modes/ModeStandby.c
    • -17
    • +0
    /firmware/App/Services/AlarmMgmt.c
    • -2
    • +0
    /firmware/App/Services/AlarmMgmt.h
    • -1
    • +0
    /firmware/App/Services/SystemCommMessages.c
2:32 pm

Hung Nguyen <hnguyen@diality.com>

DEN-11928 Updated code due to changes in DG_Fill_Mode_States in Common

Options
    • -4
    • +4
    /firmware/App/Modes/ConsumableSelfTest.c
Sunday 06 Mar 2022
8:57 pm

Dara Navaei

Align the comment.

Align the comment.

8:43 pm

Dara Navaei

Why was this include removed? Do you not need it for the treatment parameters commands?

Why was this include removed? Do you not need it for the treatment parameters commands?