firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Addressed code review comments

DEN-6890: Removed unnecessary accelerometer calibration parameter message

If no params, just remove this line from header.

If no params, just remove this line from header.

Did you add a call to this function in TaskPriority.c?

Did you add a call to this function in TaskPriority.c?

Probably don't need OperationModes.h either.

Probably don't need OperationModes.h either.

DEN-6890: Implemented first pass of DG Fluid Leak Detector driver

DG-DEN-6890_Fluid Leak Detector Driver
DG-DEN-6890_Fluid Leak Detector Driver
DEN-6890: Implemented first pass of DG Fluid Leak Detector driver

    • -2
    • +0
    ./App/Controllers/ConductivitySensors.c
    • -0
    • +255
    ./App/Controllers/FluidLeak.c
    • -0
    • +56
    ./App/Controllers/FluidLeak.h
    • -0
    • +94
    ./App/Services/SystemCommMessages.c
    • -1
    • +11
    ./App/Services/SystemCommMessages.h
DEN-5963 work in progress. tested cancellation path with water

    • -86
    • +124
    ./App/Modes/ModeHeatDisinfect.c
DEN-5963 added other inlet and outlet fans RPM

DEN-5963 tested heat disinfect

    • -102
    • +229
    ./App/Modes/ModeHeatDisinfect.c
DEN-7091: Added acid and bicarb tests

    • -14
    • +14
    ./App/Controllers/ConcentratePumps.c
    • -35
    • +63
    ./App/Controllers/ConductivitySensors.c
    • -1
    • +4
    ./App/Controllers/ConductivitySensors.h
    • -1
    • +1
    ./App/Controllers/TemperatureSensors.c
DEN-5963 worked on the heat disinfect states

    • -79
    • +121
    ./App/Modes/ModeHeatDisinfect.c
DEN-7091: Fixed FPGA revision

DEN-7091: Fixed drain pump conversion error when pump is off

DEN-7091: Expanded fill command to accept start/stop flag

    • -4
    • +12
    ./App/Services/SystemCommMessages.c
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Removed.

Removed.

DEN-7091: Addressed code review

I think setActiveReservoirCMD() sets activeReservoir.data for you. Redundant.

I think setActiveReservoirCMD() sets activeReservoir.data for you. Redundant.

Due to changes from non-volatile data management, this message is no longer available. Need to remove to fix build.

Due to changes from non-volatile data management, this message is no longer available.
Need to remove to fix build.

Why removed?

Why removed?

DEN-5963 added Ilimit to the PIControllers.c, added feedback temperature sensor to the heaters.c, worked on heat disinfect mode

DEN-5963 worked on RO pump and the pre heat disinfect states

DEN-5963 merged staging. Worked on RO pump

    • -107
    • +182
    ./App/Services/Reservoirs.c
    • -17
    • +88
    ./App/Services/SystemCommMessages.c
    • -4
    • +17
    ./App/Services/SystemCommMessages.h
DEN-5963 worked on RO pump and the drain states of heat disinfect

DG-DEN-7091_Pre Treatment Mode
DG-DEN-7091_Pre Treatment Mode
DEN-7091: Changed startup active reservoir to 1