firmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-7091: Addressed code review

Let's update to v0.5.0.

Let's update to v0.5.0.

This init is called from main at startup. This seems to work, but I'd prefer to select initial active reservoir and set reservoir valves accordingly after POST - maybe in transition to standby mode...

This init is called from main at startup. This seems to work, but I'd prefer to select initial active reservoir and set reservoir valves accordingly after POST - maybe in transition to standby mode. This way, if POST does something to the valves to test, we will set them afterward. Also gives valve driver and FPGA interface a chance to get started before we start giving actuator commands.

DEN-7091: Fixed load cell small filter index

Merge branch 'DEN-7091-pre-treatment-mode' into develop

DEN-7091: Changed function call to actual valve settings

DEN-7091: fix to last.

DEN-7091: Updated load cell filtering.

DEN-7091: Removed unused variables

    • -1
    • +1
    ./App/Controllers/ConductivitySensors.c
Merge branch 'DEN-5963-dg-heat-disinfection-2-of-2' into develop

    • -13
    • +70
    ./App/Services/SystemCommMessages.c
    • -1
    • +12
    ./App/Services/SystemCommMessages.h
Merge branch 'DEN-7091-pre-treatment-mode' of ssh://dvm-linux02:7999/dg/dgfirmware into DEN-7091-pre-treatment-mode

    • -0
    • +19
    ./App/Services/SystemCommMessages.c
DEN-7091: removed load cell filter test code.

    • -19
    • +0
    ./App/Services/SystemCommMessages.c
DEN-7091: Added build switches to disable mixing and water quality checks

    • -0
    • +2
    ./App/Controllers/ConductivitySensors.c
    • -0
    • +2
    ./App/Controllers/TemperatureSensors.c
DEN-7091: Added build switch to disable dialysate checks

    • -0
    • +8
    ./App/Controllers/ConductivitySensors.c
Shouldn't there be a few of these disinfect times? R1, R2, etc...

Shouldn't there be a few of these disinfect times? R1, R2, etc...

Delete blank line.

Delete blank line.

Delete blank line.

Delete blank line.

If we don't need this check anymore, just delete it.

If we don't need this check anymore, just delete it.

If we're not doing all of this commented out monitoring, just delete it.

If we're not doing all of this commented out monitoring, just delete it.

Why commented out here? If you don't want this disabled, comment the build switch out in DGCommon.h.

Why commented out here? If you don't want this disabled, comment the build switch out in DGCommon.h.

Why banner deleted?

Why banner deleted?

Why is this banner deleted?

Why is this banner deleted?

Why commented out?

Why commented out?

Why remove this check?

Why remove this check?

What is the purpose of this? If there is a problem, shouldn't there be an alarm/fault? Seems like current mode may not be aware that the drain pump has been stopped here.

What is the purpose of this? If there is a problem, shouldn't there be an alarm/fault? Seems like current mode may not be aware that the drain pump has been stopped here.

Not sure what's happening here. Restore this as it was.

Not sure what's happening here. Restore this as it was.

Why is this check commented out? Even if our RPM conversion is not great, we still need to be able to distinguish between off and on. Set a threshold and keep this check.

Why is this check commented out? Even if our RPM conversion is not great, we still need to be able to distinguish between off and on. Set a threshold and keep this check.

Why is all of this commented out? Just build switch the checks that aren't working and leave everything else alone.

Why is all of this commented out? Just build switch the checks that aren't working and leave everything else alone.

If this check isn't working, make a build switch to exclude them. What is the plan for this check? Can we even do it? If not, remove.

If this check isn't working, make a build switch to exclude them. What is the plan for this check? Can we even do it? If not, remove.

Fixed.

Fixed.