Activity hdfirmware/firmware/App/Modes

12:18 pm

Darren Cox <darren_m@cox.net>

More
fae0f1d
changed 2 files
DEN-10155-treatment-end-paused-state-not-displaying-resume-option, DEN-13154-heparin-syringe-empty-position-incorrect and 282 more HD-DEN-14561-1

DEN-14561 Updated for code review comments.

- -1
- +0
./ModePreTreat.c
- History
- Source
- Diff
./ModePreTreat.c
… 1 more file in changeset.

2:51 pm

Sean Nash

More
9e1b9e3
changed 7 files
DEN-10155-treatment-end-paused-state-not-displaying-resume-option, DEN-13154-heparin-syringe-empty-position-incorrect and 283 more HD-DEN-14749-1

DEN-14749: Reset preload flag in standby mode instead of after retract/seek/prime operations. Increase alarm threshold for FMD vs. LC check in wet s.t. from 5% to 10%. Refactor alarm audio and 1 minute min tx time config flags. Removed test code in reservoir code.

- -0
- +1
./ModeStandby.c
- History
- Source
- Diff
./ModeStandby.c
- -15
- +14
./ModeTreatment.c
- History
- Source
- Diff
./ModeTreatment.c
- -1
- +1
./SelfTests.c
- History
- Source
- Diff
./SelfTests.c
… 4 more files in changeset.

6:37 pm

Sean Nash

More
Re: HD-DEN-14561-1
commented on review

This is not going to work. Revisit. See code review for DEN-14749 for correct arrangement.

6:15 pm

Sean Nash

More
HD-DEN-14549-1
Closed

3:05 pm

Darren Cox <darren_m@cox.net>

More
e4f34f8
changed 1 file
DEN-10155-treatment-end-paused-state-not-displaying-resume-option, DEN-13154-heparin-syringe-empty-position-incorrect and 282 more HD-DEN-14561-1

DEN-1561 Fix integrated volume calculation for reservoir transfers.

- -5
- +2
./SelfTests.c
- History
- Source
- Diff
./SelfTests.c

3:01 pm

Dara Navaei

More
HD-DEN-14749-1
started review

HD-DEN-14749_Syringe Pump Plunger Remains Extended During Pretreatment Causing HD Software Fault

3:01 pm

Dara Navaei

More
HD-DEN-14763-1
started review

HD-DEN-14763_DG HD Dev HD DG Dvt Update Part 10

10:59 am

wbracken (deleted user)

More
Re: HD-DEN-14561-1
replied to jtaylor (deleted user)

RESOLVED IN CODE WALKTHROUGH

9:47 am

Sean Nash

More
Re: HD-DEN-14561-1
commented on review

This math no longer works because we changed the displacement time from 1 minute (which allowed for this simplified approach) to 30 seconds. Generalize the math. Suggested approach: -integrate vol...

This math no longer works because we changed the displacement time from 1 minute (which allowed for this simplified approach) to 30 seconds.
Generalize the math.

Suggested approach:
-integrate volume above properly (don't just sum rates over time).
-Should be: fmdIntegratedVolume += ( getMeasuredDialInFlowRate() / (F32)( ( SEC_PER_MIN * MS_PER_SECOND ) / TASK_GENERAL_INTERVAL ) );
-Then we don't need to convert here at end of displacement - can just use fmdIntegratedVolume as is in verify state.

9:47 am

Sean Nash

More
Re: HD-DEN-14561-1
commented on review

This simplified equation works because we added up flow (mL/min) measurements for 1 minute. If we change displacement time, it falls apart (not generalized) which happens in second displacement bel...

This simplified equation works because we added up flow (mL/min) measurements for 1 minute. If we change displacement time, it falls apart (not generalized) which happens in second displacement below (now that transfer time was reduced to 30 sec).
Let's generalize this math to clarify it and make it work regardless of time.

Suggested approach:
-integrate volume above properly (don't just sum rates over time).
-Should be: fmdIntegratedVolume += ( getMeasuredDialInFlowRate() / (F32)( ( SEC_PER_MIN * MS_PER_SECOND ) / TASK_GENERAL_INTERVAL ) );
-Then we don't need to convert here at end of displacement - can just use fmdIntegratedVolume as is in verify state.

5:16 pm