Activity dgfirmware/

6:01 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

More
e9f4774
changed 1 file
DEN-13154-heparin-syringe-empty-position-incorrect, DEN-13946-certain-temperature-sensors-trigger-wrong-alarm and 167 more DG-DEN-13946-1

Merge branch 'staging' into DEN-13946-certain-temperature-sensors-trigger-wrong-alarm

- -7
- +14
/firmware/App/Controllers/LoadCell.c
- History
- Source
- Diff
/firmware/App/Controllers/LoadCell.c

5:59 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

More
b44ada6
changed 1 file
DEN-13154-heparin-syringe-empty-position-incorrect, DEN-13946-certain-temperature-sensors-trigger-wrong-alarm and 167 more DG-DEN-13946-1

DEN-13946 - Use typedefs for iterating over sensors.

- -3
- +1
/firmware/App/Controllers/LoadCell.c
- History
- Source
- Diff
/firmware/App/Controllers/LoadCell.c

3:04 pm

Dara Navaei

More
5aee05d
changed 11 files
DEN-13050-vpi-does-not-close-after-sample-water-returns-to-flush-idle, DEN-13154-heparin-syringe-empty-position-incorrect and 171 more DG-DEN-14497-1

DEN-14497 updated the drain pump flow control algorithm. Fixed the inlet water pressure, temperature, conductivity alarm recovery in treatment modes. Updated chemical disinfect flush mode

- -16
- +27
/firmware/App/Controllers/ConductivitySensors.c
- History
- Source
- Diff
/firmware/App/.../ConductivitySensors.c
- -11
- +9
/firmware/App/Controllers/DrainPump.c
- History
- Source
- Diff
/firmware/App/Controllers/DrainPump.c
- -1
- +1
/firmware/App/Controllers/DrainPump.h
- History
- Source
- Diff
/firmware/App/Controllers/DrainPump.h
- -9
- +19
/firmware/App/Controllers/Pressures.c
- History
- Source
- Diff
/firmware/App/Controllers/Pressures.c
- -4
- +14
/firmware/App/Controllers/TemperatureSensors.c
- History
- Source
- Diff
/firmware/App/.../TemperatureSensors.c
- -145
- +117
/firmware/App/Modes/ModeChemicalDisinfect.c
- History
- Source
- Diff
/firmware/App/.../ModeChemicalDisinfect.c
- -193
- +157
/firmware/App/Modes/ModeChemicalDisinfectFlush.c
- History
- Source
- Diff
/firmware/.../ModeChemicalDisinfectFlush.c
- -8
- +2
/firmware/App/Modes/ModeFill.c
- History
- Source
- Diff
/firmware/App/Modes/ModeFill.c
- -45
- +61
/firmware/App/Modes/ModeHeatDisinfect.c
- History
- Source
- Diff
/firmware/App/Modes/ModeHeatDisinfect.c
- -0
- +2
/firmware/App/Services/AlarmMgmt.c
- History
- Source
- Diff
/firmware/App/Services/AlarmMgmt.c
- -3
- +3
/firmware/App/Services/PIControllers.c
- History
- Source
- Diff
/firmware/App/Services/PIControllers.c

1:03 pm

Bill Bracken <bill@criticalsoftwaresolutions.com>

More
c0ef736
changed 4 files
DEN-13154-heparin-syringe-empty-position-incorrect, DEN-13946-certain-temperature-sensors-trigger-wrong-alarm and 167 more DG-DEN-13946-1

DEN-13946 Use typedefs for itterating over sensors.

- -4
- +4
/firmware/App/Controllers/LoadCell.c
- History
- Source
- Diff
/firmware/App/Controllers/LoadCell.c
- -4
- +3
/firmware/App/Controllers/Pressures.c
- History
- Source
- Diff
/firmware/App/Controllers/Pressures.c
- -6
- +6
/firmware/App/Controllers/TemperatureSensors.c
- History
- Source
- Diff
/firmware/App/.../TemperatureSensors.c
- -2
- +1
/firmware/App/Controllers/Thermistors.c
- History
- Source
- Diff
/firmware/App/Controllers/Thermistors.c

11:37 am

wbracken (deleted user)

More
Re: DG-DEN-13946-1
replied to Sean Nash

Similar code is in other monitoring functions. monitorTemperatureSensors, monitorPressureSensors iterate over their list of sensors using U08. monitorThermistors iterates over thermistors using typ...

Similar code is in other monitoring functions. monitorTemperatureSensors, monitorPressureSensors iterate over their list of sensors using U08. monitorThermistors iterates over thermistors using typedef THERMISTORS_TEMP_SENSORS_T. Changed load cells to use U08 to be consistent with temperature and pressure monitoring.

Should probably change temperature, pressure and load cells to use correct typedef.

Also, it looks like why the typedefs weren't used was in order to set the initial value of the iterator to 0 (i.e, the first sensor in the list). Seems like there should be a value that is the first item in the list, i.e. TEMPSENSORS_FIRST_SENSOR.

11:07 am