hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

Override should be OR'd with timeout only. Used should have to be true even if override > 0.

Override should be OR'd with timeout only. Used should have to be true even if override > 0.

Remove blank line.

Remove blank line.

HD-DEN-16278_HD Pretreatment Prime Start State Does Not Allow Pause
HD-DEN-16278_HD Pretreatment Prime Start State Does Not Allow Pause
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14706_Need Mechanism For Alarm ID Can Message Not Acked And Alarm ID DG Can Message Not Acked
HD-DEN-14706_Need Mechanism For Alarm ID Can Message Not Acked And Alarm ID DG Can Message Not Acked
dryPressureTestsCompleted is already in this header. dryPressureTestsCompleted was added to setupForSelfTestStop() header.

dryPressureTestsCompleted is already in this header. dryPressureTestsCompleted was added to setupForSelfTestStop() header.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Add dryPressureTestsCompleted to header.

Add dryPressureTestsCompleted to header.

Updated to handle any active alarm if we havent passed pressure tests. I also modified the logic to where this valve change should only happen in d.s.p.t.

Updated to handle any active alarm if we havent passed pressure tests. I also modified the logic to where this valve change should only happen in d.s.p.t.

Fixed. Thanks!

Fixed. Thanks!

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

As you pointed out earlier, rinsebackTimerCtr is set to zero in states where BP is running, so the if condition would only be TRUE (rinsebackTimerCtr > 0) if BP is not running.

As you pointed out earlier, rinsebackTimerCtr is set to zero in states where BP is running, so the if condition would only be TRUE (rinsebackTimerCtr > 0) if BP is not running.

You removed the if (great), but I don't see that you replaced it with BP stopped check. I don't think we want a countdown if BP is running (blood not sitting).

You removed the if (great), but I don't see that you replaced it with BP stopped check. I don't think we want a countdown if BP is running (blood not sitting).

Done.

Done.

What if we got some other alarm during pressure test states? Would it be better to check dryPressureTestsCompleted?

What if we got some other alarm during pressure test states? Would it be better to check dryPressureTestsCompleted?

Put constant on other side of comparison operator. We only put constant first when using "==".

Put constant on other side of comparison operator. We only put constant first when using "==".

This if will prevent countdown timer from being displayed unless we are paused or stopped after rinseback. I think countdown timer is definitely needed for rinseback init state (pump is stopped and...

This if will prevent countdown timer from being displayed unless we are paused or stopped after rinseback. I think countdown timer is definitely needed for rinseback init state (pump is stopped and rinseback hasn't even started). May be needed for new reconnect patient state as well.
I think it might be simpler/clearer to replace this if with a check for whether BP is running or not. If BP is stopped, we proceed to next if. If BP is running, keep timeouts at zero.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

HD-DEN-16164_FW Pre Treatment Pressure Test Failures Vent TO Saline Bag
HD-DEN-16164_FW Pre Treatment Pressure Test Failures Vent TO Saline Bag