HD-DEN-15741

HD-DEN-15741

Activity on this page:

Line History

Line history chart not available

Line history chart not available

 

Stats

Commits this week: 0
Total Commits: 3,304
Last Commit: 16 Dec
Commits this week: 0

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
 
Tuesday 08 Aug 2023
9:21 am

Sean Nash

Monday 07 Aug 2023
10:21 am

wbracken  (deleted user)

Friday 04 Aug 2023
3:49 pm

Sean Nash

3:48 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

1:26 pm

jtaylor  (deleted user)

Wouldn't we want to stop if an alarm occurred during recirculation, which affected the blood circuit? My inclination would be to manage which alarms were enabled in this state rather than not imple...

Wouldn't we want to stop if an alarm occurred during recirculation, which affected the blood circuit? My inclination would be to manage which alarms were enabled in this state rather than not implement the mechanism to stop the pumps.

Implementing the change.
Done.

1:00 pm

Sean Nash

Should blood recirc blocked be based on alarm properties (as you've written here) or based on whether user stopped (specifically)? My feeling is that since there is no blood in blood side of circui...

Should blood recirc blocked be based on alarm properties (as you've written here) or based on whether user stopped (specifically)? My feeling is that since there is no blood in blood side of circuit (because rinseback was completed before we got to recirc), the NoBloodRecirc alarm property maybe doesn't apply here and we should only stop BP if the user stop alarm is active.

12:57 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:57 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:55 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:55 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:55 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:50 pm

Sean Nash

We already know dialysateRecircBlocked != TRUE here - should this just be a simple "else"?

We already know dialysateRecircBlocked != TRUE here - should this just be a simple "else"?

12:50 pm

jtaylor  (deleted user)

Done.

Done.

12:47 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

12:47 pm

jtaylor  (deleted user)

Done.

Done.

11:00 am

jtaylor  (deleted user)

Done.

Done.

10:47 am

jtaylor  (deleted user)

Done.

Done.

10:35 am

jtaylor  (deleted user)

Done.

Done.

10:03 am

Sean Nash

Keep resume behavior as it was unless Systems is telling you current behavior is wrong. recircResumeRequested flag gets set to TRUE somewhere else and I think that is the only place where the flag ...

Keep resume behavior as it was unless Systems is telling you current behavior is wrong. recircResumeRequested flag gets set to TRUE somewhere else and I think that is the only place where the flag should be set. I see no reason why it should be set here.

9:57 am

jtaylor  (deleted user)

Done.

Done.

9:40 am

jtaylor  (deleted user)

The user stop alarm drops the recirc state machine into the stopped state (stop alarm property is true). To full resume requires that "OK" is pressed, then "Resume", to continue with recirculation....

The user stop alarm drops the recirc state machine into the stopped state (stop alarm property is true). To full resume requires that "OK" is pressed, then "Resume", to continue with recirculation. Is that the behavior we want, or do we want to continue with only the "OK" acknowledgment?

Thursday 03 Aug 2023
4:45 pm

Sean Nash

Resume requested flag gets set elsewhere (from user clearing alarm -> CAN -> AlarmMgmt -> Modes -> this module) and should not be set here.

Resume requested flag gets set elsewhere (from user clearing alarm -> CAN -> AlarmMgmt -> Modes -> this module) and should not be set here.

4:36 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

4:35 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

4:34 pm

wbracken  (deleted user)

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

4:31 pm

Sean Nash

So this is only going to stop pumps/heater if both block recircs are active. I don't think that's what you want. My understanding of what you're trying to do here is this: 1) if dialysate recirc is...

So this is only going to stop pumps/heater if both block recircs are active. I don't think that's what you want.
My understanding of what you're trying to do here is this:
1) if dialysate recirc is blocked by an active alarm (includes user stop alarm), we should stop DPi and HR. Otherwise, we should run DPi and HR.
2) if user stop alarm is active or we are in any state other than recirc state, we should stop BP. Otherwise, we should run BP to recirculate blood side of circuit.

4:23 pm

Sean Nash

I don't think you need to repeat @details Inputs: and @details Outputs:. Only need those for first row. See other functions for example.

I don't think you need to repeat @details Inputs: and @details Outputs:. Only need those for first row. See other functions for example.

3:28 pm

Sean Nash

Blank line before banner comment.

Blank line before banner comment.

3:24 pm

Sean Nash

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

3:24 pm

Sean Nash

Still more here than should be. Trimmer heater is being turned off twice now. DPo should already be stopped. And VBA/VBV should be handled by state machine I would think unless user stop requiremen...

Still more here than should be. Trimmer heater is being turned off twice now. DPo should already be stopped. And VBA/VBV should be handled by state machine I would think unless user stop requirement talks about valves too - and even if so, valves should only close when user stop alarm is active - nothing to do with recirc properties in alarms.