RESOLVED IN CODE WALKTHROUGH
0.0F, maybe a #define?
Update header
Still wrong though.
RESOLVED in CODE WALKTHROUGH.
Thanks for catching that, fixed.
Corrected
Do we really want to use a whole flash sector for a flag? I thought bootloader was going to talk to UI to determine whether an update was in progress?
Comments appear to be swapped.
They should remain. Will be uncommented when all DVT units have updated hardware.
Are these alarms intentionally commented? Should they be activated or removed?
This is not user code. Next regen will restore this code. Do we really need to disable ECC test code?
Also reject if in treatment stop state and recovering from blood leak alarm.
Removed
i agree
Fixed.
Alarm should be handled inside of setAirPumpState().
Done.
I think we can just say "if ( TRUE == hasPowerBeenLost )" here. Zero trigger persistence time now that CPLD is making determination.
RESOLVED. clossing.
I think we can fit this whole comment on the 1 line. Bring closer to break; if need more room.
the application branch has no code change the changes are made on the ui.script this CR will be closed.
Remove this method of determine whether AC power has been lost. CPLD function above is what we want.
We want to use CPLD for this. Keep this line, remove line below where initialized to FALSE.
Should be dgPOSTPassed.
No longer needed?
Done
Update header.
We did not check for the upper limit in the past, does this add new rejection scenarios
Should this be a constant?
RESOLVED
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.
