firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-16164: Removed reservoir weight check in fill settle state

DEN-14706: HD pending ACK override message handling and implementation.

    • -0
    • +32
    ./App/Services/SystemCommMessages.c
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
DEN-16164: code review comments.

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!

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

DEN-16164: compiler fixes.

DEN-16164: Code review comments. Changed stop state for handle all alarms for depressurized Inverse logic for setting VBA to Pos B during dry self test failures and ensure HD only does it during dry self test.

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

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

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

Merge remote-tracking branch 'origin/DEN-15630-vtchd-598-transitions-not-found' into develop

    • -2
    • +35
    ./App/Services/SystemCommMessages.c
DEN-15630-vtchd-598-transitions-not-found

Added code in pre-treatment prime state to allow pause in start state.

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).

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

DEN-15994-rinseback-5-minute-timer-does-not-count-down

Simplification.

DEN-15994-rinseback-5-minute-timer-does-not-count-down

Updated for CR comments.