dgfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Merge branch 'DEN-15455-fw-dvt-6-and-16-cap-switches-do-not-update-when-caps-close' into develop

DEN-15455 updated chem disinfect and chem flush disinfect to not clear the acid insert and remove alarms

DEN-15342 addressed the heat disinfect active cool state skip

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

DEN-15329 added a minimum check for the RO pump duty cycle when it is in the control to pressure state

Add alarmDetectedPendingTrigger to header.

Add alarmDetectedPendingTrigger to header.

DEN-15404 added doxygen comments to the RO permeate sample mode

    • -3
    • +10
    /firmware/App/Modes/ModeChemicalDisinfect.c
    • -32
    • +170
    /firmware/App/Modes/ModeROPermeateSample.c
DG-DEN-15455_FW Dvt 6 And 16 Cap Switches DO Not Update When Caps Close - Ready
DG-DEN-15455_FW Dvt 6 And 16 Cap Switches DO Not Update When Caps Close - Ready
DG-DEN-15477_Add Windowed Timer TO Drain Pump Encoder Error
DG-DEN-15477_Add Windowed Timer TO Drain Pump Encoder Error
Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Merge branch 'DEN-15455-fw-dvt-6-and-16-cap-switches-do-not-update-when-caps-close' into develop

DEN-15455, DEN-15471 fixed the chemical disinfect caps monitor

    • -3
    • +10
    /firmware/App/Modes/ModeChemicalDisinfect.c
Fix alignment.

Fix alignment.

DEN-15404 work in progress. Worked on the RO permeat sample mode

    • -14
    • +711
    /firmware/App/Modes/ModeROPermeateSample.c
    • -0
    • +11
    /firmware/App/Modes/ModeROPermeateSample.h
Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Merged staging

    • -9
    • +27
    /firmware/App/Controllers/Heaters.c
DG-DEN-13989_FW Concentrate Pump Cp1 Speed Error Observed
DG-DEN-13989_FW Concentrate Pump Cp1 Speed Error Observed
Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

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

Merged DEN-15462

Yes. The pump minimum target is 3, attempting to set smaller than that limit results in the pump being turned off. The underlying issue is that stepConcentratePumpToTargetSpeed does not initialize...

Yes. The pump minimum target is 3, attempting to set smaller than that limit results in the pump being turned off.

The underlying issue is that stepConcentratePumpToTargetSpeed does not initialize speedIncrease or hasTgtBeenReached, or update the currentPumpSpeed target unless the currentToTargetDiff is < nearly zero. Since the speeds are quantized, we'll see zero more frequently than expected. Particularly when stopping the pump, the Diff will be zero, and currentPumpSpeed may never be updated, leading to odd transient conditions.

By increasing the "zero" level, and adding the initialization and else clauses for zero error and stopping the pump, we guarantee that the speed setting is always set intentionally. Also we already have a minimum speed parameter, used to determine when to stop the motor, and attempts at motor control at these low levels.

DEN-15462 added to be able to disable the voltages monitor with the software configurations

DG-DEN-15462_Nelson Support With Alarm 77
DG-DEN-15462_Nelson Support With Alarm 77
DEN-15367, DEN-15404 updated the code

    • -0
    • +10
    /firmware/App/Modes/ModeROPermeateSample.c