hdfirmware / firmware / App

Services

Clone Tools

Stats

Commits this week: 0
Total Committers: 16
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
Wednesday 18 Jan 2023
3:34 pm

Dara Navaei

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

Options
    • -0
    • +12
    ./AlarmMgmt.c
    • -0
    • +1
    ./AlarmMgmt.h
    • -0
    • +12
    ./WatchdogMgmt.c
    • -1
    • +2
    ./WatchdogMgmt.h
  5. … 10 more files in changeset.
1:06 pm

Sean Nash

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

Options
    • -5
    • +7
    ./SystemCommMessages.c
    • -2
    • +2
    ./SystemCommMessages.h
  3. … 2 more files in changeset.
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
    • -1
    • +38
    ./AlarmMgmt.c
    • -0
    • +1
    ./AlarmMgmt.h
    • -0
    • +1
    ./AlarmMgmtSWFaults.h
  4. … 5 more files in changeset.
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: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?

Tuesday 17 Jan 2023
2:58 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

2:57 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

2:56 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

2:25 pm

dcox  (deleted user)

This config check is unnecessary and should be removed. The vars should always be initialized. Enable/Disable Alarms are done when checking the direction, etc.

This config check is unnecessary and should be removed. The vars should always be initialized. Enable/Disable Alarms are done when checking the direction, etc.

1:05 pm

James Walter Taylor <jtaylor@diality.com>

HD-DEN-14561_Release Preprocessor Flag Specifies Code That Conflicts With SW Config Debug

Corrected _RELEASE_ logic, release build executes the same block as non debug selection

Options
    • -10
    • +11
    ./AlarmMgmt.c
  2. … 3 more files in changeset.
1:04 pm

Sean Nash

They're kind of different. The stop function is fairly straightforward and is part of syringe pump driver. The start function is a lot more complex as it needs to figure out what it should be doing...

They're kind of different. The stop function is fairly straightforward and is part of syringe pump driver. The start function is a lot more complex as it needs to figure out what it should be doing and is part of treatment mode module.

12:13 pm

dcox  (deleted user)

Inconsistent naming here. StopSyringePump, StartHeparinPump. Suggest using same name for both.

Inconsistent naming here. StopSyringePump, StartHeparinPump. Suggest using same name for both.

11:35 am

Sean Nash

Done.

Done.

11:32 am

Sean Nash

Done.

Done.

11:32 am

Sean Nash

Done.

Done.

9:57 am

wbracken  (deleted user)

Add pressureSelfTestNormalizedStartTime to header.

Add pressureSelfTestNormalizedStartTime to header.

9:55 am

wbracken  (deleted user)

Add pressureSelfTestPreNormalStartTime to header.

Add pressureSelfTestPreNormalStartTime to header.

9:54 am

wbracken  (deleted user)

Add pressureSelfTestPreNormalStartTime to header.

Add pressureSelfTestPreNormalStartTime to header.

9:44 am

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

9:30 am

Sean Nash

DEN-14549: Misc updates to dry self-tests in pre-treatment.

Options
    • -1
    • +1
    ./SystemComm.c
    • -8
    • +3
    ./SystemCommMessages.c
  3. … 2 more files in changeset.
Monday 16 Jan 2023
3:01 pm

Dara Navaei

HD-DEN-14561_Release Preprocessor Flag Specifies Code That Conflicts With SW Config Debug
HD-DEN-14561_Release Preprocessor Flag Specifies Code That Conflicts With SW Config Debug
Saturday 14 Jan 2023
2:15 pm

Sean Nash

DEN-14549: Fixed issue where door open alarm would have condition cleared repeatedly in standby start state. Made top alarm determination done from scratch each time.

Options
    • -8
    • +10
    ./AlarmMgmt.c
  2. … 1 more file in changeset.
1:30 pm

Sean Nash

DEN-14549: Removed obsolete internal ADC channel. Updates internal ADC channel transfer functions. Misc pre-tx state optimizations.

Options
    • -3
    • +2
    ./AlarmMgmt.c
    • -0
    • +1
    ./AlarmMgmtSWFaults.h
  3. … 6 more files in changeset.
Friday 13 Jan 2023
8:46 am

Sean Nash

It is in the header under Outputs.

It is in the header under Outputs.