dd-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-3958 disabling few alarms

LDT-3958 alarm disabling

Added new field for d1 filtered temperature and restored the pressure sensor temperature.

Added new field for d1 filtered temperature and restored the pressure sensor temperature.

The latest code from staging has an update and fixes this comment. It's not showing here, since the file was already reviewed.

The latest code from staging has an update and fixes this comment. It's not showing here, since the file was already reviewed.

as discussed, these were outcome of the test results. we will review these gains with latest controller.

as discussed, these were outcome of the test results. we will review these gains with latest controller.

Done.

Done.

Done.

Done.

LDT-3223-dialysate-flow-rate---dd---mode-tel-dev-test-3-30-2026 : Added testcases to TaskGeneral

LDT-3223-dialysate-flow-rate---dd---mode-tel-dev-test-3-30-2026 : Added test cases for PersistentAlarm.c file

LDT-3958 heater control update

    • -15
    • +1
    /firmware/App/Controllers/Heaters.c
LDT-3958 resolving merge issues

    • -10
    • +31
    /firmware/App/Modes/ModeGenDialysate.c
LEAHI-DD-FIRMWARE-LDT-3975_Water Treatment and Degassing - FW Iteration - 4/4:
LEAHI-DD-FIRMWARE-LDT-3975_Water Treatment and Degassing - FW Iteration - 4/4:
LDT-3975: Added distinct s/w fault IDs for PID controller faults.

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

Resolved the conflicts from staging

Merged staging and resolved conflicts

    • -2
    • +2
    /firmware/App/Modes/ModeGenDialysate.c
Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

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

Merged LDT-3272

On DD f/w side, this should just be a simple pump controller. States should be Off and On (control to commanded Qsub). There is no flow sensor for feedback control, so assuming this is open loop si...

On DD f/w side, this should just be a simple pump controller. States should be Off and On (control to commanded Qsub). There is no flow sensor for feedback control, so assuming this is open loop similar to UF pump.
GenDialysate mode will have an HDF sub-state that signals this controller from Off to On and would also call separate controller for heater and separate (Switches) monitor for door sensors.

With all of that in mind, I think this unit should have a simpler name like SubPump.

I think there should be some state monitoring for potential alarm(s) in here too. Similar to BP door on TD f/w side where the current mode/state can signal this unit that it expects specific switch...

I think there should be some state monitoring for potential alarm(s) in here too. Similar to BP door on TD f/w side where the current mode/state can signal this unit that it expects specific switch state(s) (e.g. while in HDF state, we would expect the cap to be open and stowed and we would want to alarm if that was not the case).

Remove extra blank line.

Remove extra blank line.

LDT-3272 addressed the code review comments

    • -16
    • +0
    /firmware/App/Services/SystemCommDD.h
Where is call to HDF controller?

Where is call to HDF controller?

Monitors should come before mode mgmt above.

Monitors should come before mode mgmt above.

What about HemoDiaFiltration.h?

What about HemoDiaFiltration.h?

What about initHDF?

What about initHDF?

Where is HemoDiaFiltration.h?

Where is HemoDiaFiltration.h?