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 minutes ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Thursday 19 Jan 2023
10:59 am

Michael Garthwaite

DEN-14414: Implemented sub state events.

Options
    • -0
    • +6
    /firmware/App/Modes/ConsumableSelfTest.c
    • -0
    • +19
    /firmware/App/Modes/Dialysis.c
    • -0
    • +13
    /firmware/App/Modes/ModePostTreat.c
    • -0
    • +6
    /firmware/App/Modes/ModeTreatmentParams.c
    • -0
    • +6
    /firmware/App/Modes/PreTreatmentRecirc.c
    • -0
    • +6
    /firmware/App/Modes/Prime.c
    • -1
    • +5
    /firmware/App/Modes/Rinseback.c
    • -0
    • +7
    /firmware/App/Modes/SampleWater.c
    • -0
    • +19
    /firmware/App/Modes/SelfTests.c
    • -0
    • +5
    /firmware/App/Modes/TreatmentEnd.c
    • -0
    • +5
    /firmware/App/Modes/TreatmentRecirc.c
    • -0
    • +5
    /firmware/App/Modes/TreatmentStop.c
10:55 am

dcox  (deleted user)

Updated for only reservoir 2 fill state.

Updated for only reservoir 2 fill state.

10:53 am

Darren Cox <darren_m@cox.net>

DEN-14561 Change DPi fast flow only to Reservoir 2 fill.

Options
    • -2
    • +3
    /firmware/App/Modes/Prime.c
10:47 am

wbracken  (deleted user) and Sean Nash

9:27 am

Sean Nash

There are two dialysate prime states. First goes through dialyzer and both DPs are running and we still want 300 rate. Second bypasses dialyzer and only DPi is running and we want 600 rate.

There are two dialysate prime states. First goes through dialyzer and both DPs are running and we still want 300 rate. Second bypasses dialyzer and only DPi is running and we want 600 rate.

9:26 am

Sean Nash

This looks like we're skipping draining of reservoirs before doing any filling if priming disabled. I think we always want to drain both reservoirs first. Also, if we go to fill state like this, wi...

This looks like we're skipping draining of reservoirs before doing any filling if priming disabled. I think we always want to drain both reservoirs first.
Also, if we go to fill state like this, will it do a flush or a normal fill? I think it will do a flush fill since flushed flag is still FALSE.

9:19 am

Sean Nash

Remove extra blank line.

Remove extra blank line.

9:17 am

Sean Nash

We should initialize these with a loop iterating through each reservoir (in case a reservoir is added/removed at some point).

We should initialize these with a loop iterating through each reservoir (in case a reservoir is added/removed at some point).

9:16 am

Sean Nash

This should probably be 600 since we'll only be transferring 300 to R1 in wet self-tests.

This should probably be 600 since we'll only be transferring 300 to R1 in wet self-tests.

9:10 am

Sean Nash

DEN-14549: Rolled back change to include blood prime for normal inline pressure range checks. Leave as fixed max only range check in blood prime.

Options
    • -4
    • +2
    /firmware/App/Controllers/PresOccl.c
9:02 am

wbracken  (deleted user)

Alignment

Alignment

Wednesday 18 Jan 2023
5:36 pm

James Walter Taylor <jtaylor@diality.com>

HD-DEN-14561

Extended changes that correct RELEASE flag logic

Provide multiple flush passes in pretreatment

And eliminate all flushes when priming is disabled

Options
    • -48
    • +43
    /firmware/App/Modes/ModePreTreat.c
    • -1
    • +1
    /firmware/App/Modes/SelfTests.c
4:23 pm

Dara Navaei

Merge branch 'staging' into DEN-14559-dg_hd_dev-hd_dg_dvt-update-part-8

Options
    • -3
    • +7
    /firmware/App/Controllers/Valves.c
3:34 pm

Dara Navaei

DEN-14559 updated the code to be able to run POST requested by the Dialin user

Options
    • -0
    • +12
    /firmware/App/Controllers/AlarmLamp.c
    • -1
    • +2
    /firmware/App/Controllers/AlarmLamp.h
    • -0
    • +12
    /firmware/App/Controllers/Buttons.c
    • -1
    • +2
    /firmware/App/Controllers/Buttons.h
    • -0
    • +13
    /firmware/App/Controllers/Valves.c
    • -0
    • +2
    /firmware/App/Controllers/Valves.h
    • -0
    • +12
    /firmware/App/Drivers/SafetyShutdown.c
    • -0
    • +1
    /firmware/App/Drivers/SafetyShutdown.h
    • -3
    • +15
    /firmware/App/Modes/ModeInitPOST.c
    • -1
    • +1
    /firmware/App/Modes/OperationModes.c
    • -0
    • +12
    /firmware/App/Services/AlarmMgmt.c
    • -0
    • +1
    /firmware/App/Services/AlarmMgmt.h
    • -0
    • +12
    /firmware/App/Services/WatchdogMgmt.c
    • -1
    • +2
    /firmware/App/Services/WatchdogMgmt.h
3:29 pm

Darren Cox <darren_m@cox.net>

DEN-14561 Update timing and volumesfor speeding up Pre-Treat part 1.

Options
    • -2
    • +2
    /firmware/App/Modes/ModePreTreat.c
    • -1
    • +1
    /firmware/App/Modes/Prime.c
    • -5
    • +6
    /firmware/App/Modes/SelfTests.c
1:12 pm

Sean Nash

Fixed merge conflicts from staging.

Options
    • -1
    • +2
    /firmware/App/Controllers/Valves.c
1:06 pm

Sean Nash

Merge branch 'staging' into DEN-14549-system-fault-after-resuming-heparin-then-starting-saline-bolus-during-treatment

Options
    • -12
    • +12
    /firmware/App/Modes/ModeInitPOST.c
    • -21
    • +5
    /firmware/App/Modes/SelfTests.c
    • -5
    • +7
    /firmware/App/Services/SystemCommMessages.c
    • -2
    • +2
    /firmware/App/Services/SystemCommMessages.h
12:38 pm

dcox  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

12:21 pm

Sean Nash

DEN-14549: Added function to see if an alarm condition is detected. Added a function to determine whether A/C power loss condition is detected and used them where needed.

Options
    • -2
    • +3
    /firmware/App/Controllers/BloodFlow.c
    • -2
    • +4
    /firmware/App/Controllers/DialInFlow.c
    • -1
    • +3
    /firmware/App/Controllers/DialOutFlow.c
    • -0
    • +21
    /firmware/App/Drivers/Battery.c
    • -0
    • +1
    /firmware/App/Drivers/Battery.h
    • -1
    • +38
    /firmware/App/Services/AlarmMgmt.c
    • -0
    • +1
    /firmware/App/Services/AlarmMgmt.h
    • -0
    • +1
    /firmware/App/Services/AlarmMgmtSWFaults.h
11:58 am

Sean Nash

DEN-14549: include venous check in including blood prime.

Options
    • -2
    • +5
    /firmware/App/Controllers/PresOccl.c
11:55 am

dcox  (deleted user)

Yes, may need to be done when merging to Staging. Is this flag used independently anywhere? Or planned to be?

Yes, may need to be done when merging to Staging. Is this flag used independently anywhere? Or planned to be?

11:53 am

dcox  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

11:51 am

Sean Nash

DEN-14549: Addressed code review comments.

Options
    • -1
    • +11
    /firmware/App/Controllers/Voltages.c
    • -1
    • +1
    /firmware/App/Drivers/Battery.c
11:50 am

Sean Nash

I don't think I have this bug fix in my branch. Needs to be addressed in a more recent branch.

I don't think I have this bug fix in my branch. Needs to be addressed in a more recent branch.

11:48 am

Sean Nash

Good catch. Updated Voltages.c to behave same as Battery.c (trigger the correction alarm depending on whether in a treatment mode).

Good catch. Updated Voltages.c to behave same as Battery.c (trigger the correction alarm depending on whether in a treatment mode).

11:15 am

dcox  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

11:13 am

dcox  (deleted user)

This check for _IN_TREATMENT may need to be added to Valves.c ALARM_ID_HD_AC_POWER_LOST check from recent bug fix. Both look to be checked in every case. Is this flag used independently anywhere?

This check for _IN_TREATMENT may need to be added to Valves.c ALARM_ID_HD_AC_POWER_LOST check from recent bug fix. Both look to be checked in every case. Is this flag used independently anywhere?

11:08 am

dcox  (deleted user)

Treatment mode check is not done in Voltages.c, so ALARM_ID_HD_AC_POWER_LOST will be set there in addition to ALARM_ID_HD_AC_POWER_LOST_IN_TREATMENT here. Is this the expected behavior?

Treatment mode check is not done in Voltages.c, so ALARM_ID_HD_AC_POWER_LOST will be set there in addition to ALARM_ID_HD_AC_POWER_LOST_IN_TREATMENT here. Is this the expected behavior?

10:17 am

Sean Nash

DEN-14549: clear treatment version of ac power loss alarm too when ac is restored. prevent pump current alarms when ac power loss in treatment is active too.

Options
    • -1
    • +1
    /firmware/App/Controllers/BloodFlow.c
    • -1
    • +1
    /firmware/App/Controllers/DialInFlow.c
    • -1
    • +1
    /firmware/App/Controllers/DialOutFlow.c
    • -0
    • +1
    /firmware/App/Drivers/Battery.c
9:57 am

Sean Nash

DEN-14549: ensure air pump is off at start of prime state in pre-tx.

Options
    • -0
    • +6
    /firmware/App/Modes/Prime.c