Dashboard

If we're not including a timestamp, can we just remove this line of code?

If we're not including a timestamp, can we just remove this line of code?

Why commented out?

Why commented out?

LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
Swap order of == for these 2 if statements - put constant on left side.

Swap order of == for these 2 if statements - put constant on left side.

LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
Why?

Why?

LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
LEAHI-DRYDEMO-LDT-5303_SW Misc & Enhc - SW - 01 - Dry-Demo - HDF - R&I
Why is UF volume being added here? If we add it, we need to change DD f/w too. But it is not clear to me why DD f/w needs to know the UF volume.

Why is UF volume being added here? If we add it, we need to change DD f/w too. But it is not clear to me why DD f/w needs to know the UF volume.

Use ternary operator to explicitly set to TRUE or FALSE.

Use ternary operator to explicitly set to TRUE or FALSE.

LDT:4049- addressed review comments

LDT:4049- addressed review comments

LDT:4049- addressed review comments

    • -11
    • +13
    /firmware/App/Modes/ModeStandby.c
Our TODO comments are all CAPs.

Our TODO comments are all CAPs.

I think these if statements should be looking for valve state == VALVE_STATE_IDLE, not <

I think these if statements should be looking for valve state == VALVE_STATE_IDLE, not <

Fix implementation to return a BOOL (TRUE if SW_COMPATIBILITY_REV == ddVersion.compatibilityRev.

Fix implementation to return a BOOL (TRUE if SW_COMPATIBILITY_REV == ddVersion.compatibilityRev.

Should return T/F. Change comment to "TRUE if DD f/w is compatible with TD f/w, FALSE if not.".

Should return T/F. Change comment to "TRUE if DD f/w is compatible with TD f/w, FALSE if not.".

Should return a BOOL (that is what ModeStandby is expecting).

Should return a BOOL (that is what ModeStandby is expecting).

Fix isDDCompatible() function so that it returns a T/F instead of a compatibility revision.

Fix isDDCompatible() function so that it returns a T/F instead of a compatibility revision.

Align comment.

Align comment.

Why are these added? And why commented out. H13 and H20 are not rotary valves - they are 3 way valves. They do not belong here.

Why are these added? And why commented out. H13 and H20 are not rotary valves - they are 3 way valves. They do not belong here.

LDT-5248 System Interface for Users - SW - 01 - Bicarb/Acid Low Replacement - R&I - 02: SDD - Design Diagram - plantuml & Review

sdd

    • binary
    /SDDs/LowBicardWorkflow.png
    • -0
    • +81
    /SDDs/lowBicarb.puml
New message ID decleartion for air pump override

Messaging ID configuration

LDT:4072-addressed review comments

    • -10
    • +17
    /firmware/App/Monitors/Battery.c
LDT:4049 Addressed review comments

    • -317
    • +11
    /firmware/App/Modes/ModeStandby.c