hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-14011: implemented proper exit condition for dry self start state.

Done.

Done.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Do we need an else where we reset 10 second start time (if latch opened)?

Do we need an else where we reset 10 second start time (if latch opened)?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Fixed. Created handleDrySelfTestStartState() that counts settle time once the latch is determined closed.

Fixed. Created handleDrySelfTestStartState() that counts settle time once the latch is determined closed.

Fixed. Thanks!

Fixed. Thanks!

Fixed. Only checking if BP is running when are in treatment mode.

Fixed. Only checking if BP is running when are in treatment mode.

DEN-14011: created handleDrySelfTestStartState. code review comments.

Updated to 600.

Updated to 600.

DEN-14561 Update PRE_TREATMENT_FILL_RESERVOIR_TWO_VOLUME_ML per code review comment.

Should this be moved into "if" body? Doesn't appear to need setting until we resume.

Should this be moved into "if" body? Doesn't appear to need setting until we resume.

Should this 10 seconds only start once latch is locked?

Should this 10 seconds only start once latch is locked?

Not sure exception for treatment stop state still makes sense. We can be running BP in that state now depending on alarm. Should still alarm on occlusion.

Not sure exception for treatment stop state still makes sense. We can be running BP in that state now depending on alarm. Should still alarm on occlusion.

So R2 fill was 1100 and you say it ended up at 500. Would be 800 if keep same at 1100. If we change to 600, would end around 300 as you say - I think that's right.

So R2 fill was 1100 and you say it ended up at 500. Would be 800 if keep same at 1100. If we change to 600, would end around 300 as you say - I think that's right.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

    • -2
    • +2
    /firmware/App/Modes/ConsumableSelfTest.c
    • -2
    • +2
    /firmware/App/Modes/ModeTreatmentParams.c
    • -2
    • +2
    /firmware/App/Modes/PreTreatmentRecirc.c
Merge branch 'DEN-14328-hd-event-issues' into staging

HD-DEN-14011_Occulsion Alarm When Reseating
HD-DEN-14011_Occulsion Alarm When Reseating
Bamboo Commit: Updated HDCommon.h with build versions from Bamboo

DEN-14011: added condition to check if BP is running while we do occlusion checks.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-14011: dry self test transitions to DRY_SELF_TESTS_START_STATE on dry self retries instead of DRY_SELF_WAIT_FOR_DOOR_CLOSE_STATE.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.