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 14 Mar 2023
5:41 pm

Sean Nash

If we get interrupted by an alarm, how will these vars get reset?

If we get interrupted by an alarm, how will these vars get reset?

5:36 pm

Sean Nash

I don't think we can just shift into reverse here. I think I had last SRS written in a way that we would go tack to the dialysate pumps off state to ramp down again before moving to reprime state.

I don't think we can just shift into reverse here. I think I had last SRS written in a way that we would go tack to the dialysate pumps off state to ramp down again before moving to reprime state.

5:34 pm

Sean Nash

I suppose that could work.

I suppose that could work.

5:28 pm

Sean Nash

Switching valves to dialyzer with DPo stopped initially will be a shock and temporarily send a lot of dialysate to the patient while DPo ramps up. So I think DPi should be hard stopped first and th...

Switching valves to dialyzer with DPo stopped initially will be a shock and temporarily send a lot of dialysate to the patient while DPo ramps up. So I think DPi should be hard stopped first and then start both DPi and DPo at fast purge rate together.

5:28 pm

Steve Jarpe

The timer should have been 5 minutes, my mistake. The purpose is to prevent repeated triggering of the reprime after the 5 minute before treatment reprime is done, as you pointed out. If I change t...

The timer should have been 5 minutes, my mistake. The purpose is to prevent repeated triggering of the reprime after the 5 minute before treatment reprime is done, as you pointed out. If I change the 5 hours to 5 minutes, is that OK?

5:06 pm

Sean Nash

Recommend removing this else if and just commanding pumps to zero when transitioning to reprime state.

Recommend removing this else if and just commanding pumps to zero when transitioning to reprime state.

5:02 pm

Sean Nash

Timer counter is reset in both cases so should be done once before or after the in progress condition is checked.

Timer counter is reset in both cases so should be done once before or after the in progress condition is checked.

5:00 pm

Sean Nash

Need to command dialysate pumps to zero flow rate so they start ramping down.

Need to command dialysate pumps to zero flow rate so they start ramping down.

4:55 pm

Sean Nash

This check to see if it's time to do reprime should be moved to the handler for dialysis: ultrafiltration state (function just below this one) - because we would not want to start a reprime in the ...

This check to see if it's time to do reprime should be moved to the handler for dialysis: ultrafiltration state (function just below this one) - because we would not want to start a reprime in the middle of a saline bolus or a reprime and because we should only be incrementing the timer counter during actual treatment (when treatment time is not paused).

4:43 pm

Sean Nash

I don't understand what the timer > 5 hours is doing here. Timer counter should never get bigger than 30 minutes (resets to zero after reprime completes), so would never get to 5 hours anyway. I th...

I don't understand what the timer > 5 hours is doing here. Timer counter should never get bigger than 30 minutes (resets to zero after reprime completes), so would never get to 5 hours anyway.
I think we just do reprime if 30 min interval OR 5 minutes remaining in treatment. And once we do the reprime for 5 min before end of treatment, we need some way to not do it again (i.e. if we just say time remaining <= 5 minutes, we will keep doing reprimes over and over again for the last 5 minutes).

4:39 pm

Sean Nash

Comment doesn't match const name. This is essentially 5 hours. Why are we repriming after 5 hours? We have a #define below that seems to be better suited for the reprime 5 minutes before end of tre...

Comment doesn't match const name.
This is essentially 5 hours. Why are we repriming after 5 hours?
We have a #define below that seems to be better suited for the reprime 5 minutes before end of treatment. So why do we need this const?

4:27 pm

Sean Nash

Do we need something similar to this for dialyzer reprime?

Do we need something similar to this for dialyzer reprime?

4:26 pm

Sean Nash

Maybe state durations instead of function delays.

Maybe state durations instead of function delays.

4:03 pm

Sean Nash

I think the rest of these are durations, not intervals.

I think the rest of these are durations, not intervals.

2:54 pm

dcox  (deleted user)

Based on your comment in Teams: @ Qd = 450 in your data set... ~8 seconds spent between drain and fill that suggests maybe our just-in-time is a bit off - maybe should have started fill immediately.

Based on your comment in Teams:
@ Qd = 450 in your data set... ~8 seconds spent between drain and fill that suggests maybe our just-in-time is a bit off - maybe should have started fill immediately.

2:39 pm

dcox  (deleted user)

Updated.

Updated.

Monday 13 Mar 2023
9:20 am

Sean Nash

Did Blaine ask for this?

Did Blaine ask for this?

Sunday 12 Mar 2023
10:05 pm

wbracken  (deleted user)

Indent for reddability?

Indent for reddability?

10:01 pm

wbracken  (deleted user)

Don't see timeReservoirInUF being used?

Don't see timeReservoirInUF being used?

Friday 10 Mar 2023
4:01 pm

Sean Nash

3:58 pm

Dara Navaei and wbracken  (deleted user)

3:41 pm

Dara Navaei

HD-DEN-14918_Pba Pbo And OB Not Available ON Debug Screen
HD-DEN-14918_Pba Pbo And OB Not Available ON Debug Screen
1:20 pm

wbracken  (deleted user)

Isn't deleted statement correct?

Isn't deleted statement correct?

1:19 pm

wbracken  (deleted user)

Isn't deleted statement correct?

Isn't deleted statement correct?

1:18 pm

wbracken  (deleted user)

Isn't deleted statement correct?

Isn't deleted statement correct?

1:17 pm

wbracken  (deleted user)

Isn't delete statement correct?

Isn't delete statement correct?

10:19 am

Sean Nash

I think that's true in calls to serialize function, but I don't think this function wants this kind of flag (i.e. it's not looking for whether to require ACK or not - it wants to know if we are ACK...

I think that's true in calls to serialize function, but I don't think this function wants this kind of flag (i.e. it's not looking for whether to require ACK or not - it wants to know if we are ACKing (TRUE) or NAKing (FALSE) this command). Here, we are NAKing the command, so FALSE was appropriate.

10:16 am

Sean Nash

Has DG check-in msg ID been made available for re-purpose? RESOLVED in CODE WALKTHROUGH.

Has DG check-in msg ID been made available for re-purpose?
RESOLVED in CODE WALKTHROUGH.

10:14 am

Michael Garthwaite

DG no longer sends this message. We determine DG check-in by receiving the DG op mode command.

DG no longer sends this message. We determine DG check-in by receiving the DG op mode command.

10:01 am

Dara Navaei

HD-DEN-14888_Unable TO Trigger Alarm 47 Ultrafiltration Rate IS Too High Error During Treatment
HD-DEN-14888_Unable TO Trigger Alarm 47 Ultrafiltration Rate IS Too High Error During Treatment