firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen

Added a space.

DEN-14999: Removed alarm 203 (dg POST adc rail check for pressure sensors). Removed alarm 116 (dg temperature sensors adc rail check). Removed adc temperature sensors from alarm 115 (dg temperature sensors range check).

    • -36
    • +1
    ./App/Controllers/TemperatureSensors.c
DEN-15016-system-faults-during-treatment-then-returns-to-the-standby-screen

Increased MSG ACK timeouts. Delayed expecting ACKs until HD turns on and is sending messages.

DG-DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen
DG-DEN-15016_System Faults During Treatment Then Returns TO The Standby Screen
Spaces inside (). And explicit condition (i.e. ( TRUE == isConvNotValid ).

Spaces inside (). And explicit condition (i.e. ( TRUE == isConvNotValid ).

Corrected

Corrected

DEN-13674, DEN-13675 changed the fill mode to be able to restart fill if the acid and bicarb conducitivies are out of range

Merge branch 'staging' into DEN-14999-alarm-table-updates

DEN-14999 added a blank line

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

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

    • -1
    • +1
    ./App/Controllers/ConductivitySensors.c
DEN14646 - Valves variable renamed.

Merge branch 'staging' into DEN-14646-fpga-alarm-properties-and-timing-3

    • -7
    • +15
    ./App/Controllers/ConductivitySensors.c
    • -1
    • +2
    ./App/Controllers/TemperatureSensors.c
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

DEN-15014 addressed the code review comment

I did. I still see a call to sendAckResponseMsg() and I don't think it should be here.

I did. I still see a call to sendAckResponseMsg() and I don't think it should be here.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-15014 addressed the code review comment

Check the diff of the last two commits.

Check the diff of the last two commits.

What I meant is that we don't need to send anything other than response.

What I meant is that we don't need to send anything other than response.

Done

Done

Remove TODO?

Remove TODO?

DEN-15014 updated the fill mode by removing the enable RO only mode variable from the init function to not be set to FALSE every time

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.