•  

Comment Results

Review Name Created Custom Fields Content
HD-DEN-14547-1 06 Feb 2023

Alphabetical ordering.

UI-DEN-14609-1 07 Feb 2023

Removed ethernet setup line.

HD-DEN-14547-1 07 Feb 2023

Fixed. Thanks!

HD-DEN-14561-1 07 Feb 2023

The normal value is too tight for the current hardware; it always fails this test, without the wider volume tolerance.

HD-DEN-14561-1 07 Feb 2023

Done (By Darren)

HD-DEN-14561-1 07 Feb 2023

Done (by Darren)

HD-DEN-14561-1 07 Feb 2023

Done.

HD-DEN-14561-1 07 Feb 2023

This works, even if the s.t. dry test is skipped.

HD-DEN-14561-1 07 Feb 2023

This turns off the trimmer hearter, to smooth heater control (as before). The switch state (next up) actually makes the reservoir switch, the next state.

HD-DEN-14561-1 07 Feb 2023

Done.

HD-DEN-14561-1 07 Feb 2023

Done.

HD-DEN-14561-1 07 Feb 2023

Done (By Darren).

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 04 Feb 2023

This is not going to work. Revisit. See code review for DEN-14749 for correct arrangement.

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 07 Feb 2023

RESOLVED IN CODE WALKTHROUGH

HD-DEN-14561-1 07 Feb 2023

Add pendingUFVolumeChange, pressTreatmentTimeSecs, pendingParamChangesTimer,

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14561-1 07 Feb 2023

RESOLVED in CODE WALKTHROUGH.

HD-DEN-14547-1 07 Feb 2023

Fixed. Thanks!

UI-DEN-14609-1 07 Feb 2023

RESOLVED

UI-DEN-14609-1 07 Feb 2023

RESOLVED

UI-DEN-14609-1 08 Feb 2023

Updated

UI-DEN-14609-1 08 Feb 2023

RESOLVED

HD-DEN-14547-1 07 Feb 2023

Comment Alignment

HD-DEN-14547-1 07 Feb 2023

Use SEC_PER_MIN instead of 60. Also comment says min, but really it is ms.

HD-DEN-14589-1 07 Feb 2023

Alignment

UI-DEN-14609-1 07 Feb 2023

This line should be in the else block.

HD-DEN-14561-1 08 Feb 2023

RESOLVED IN CODE WALKTHROUGH

HD-DEN-14561-1 08 Feb 2023

Why is this variable an int? It should be U32.

HD-DEN-14547-1 08 Feb 2023

RESOLVED IN CODE WALKTHROUGH.

DIALIN-DEN-16454-1 19 Oct 2023

These should not be commented out. Current FW builds have MSG_ID_PRESSURE_OCCLUSION_DATA = 0x9 with the following structure.

/// Payload record structure for the pressure & occlusions data message.

F32 arterialPressure; ///< Latest arterial pressure (mmHg)
F32 venousPressure; ///< Latest venous pressure (mmHg)
U32 bldPumpOcclusion; ///< Latest BP occlusion (no units)
U32 presLimitState; ///< Current pressure limits state (enum-stabilization or stable)
S32 artMinLimit; ///< Current arterial minimum pressure limit (mmHg)
S32 artMaxLimit; ///< Current arterial maximum pressure limit (mmHg)
S32 venMinLimit; ///< Current venous minimum pressure limit (mmHg)
S32 venMaxLimit; ///< Current venous maximum pressure limit (mmHg)
F32 arterialLongFilterPres; ///< Latest long filtered arterial pressure (mmHg)
F32 venousLongFilterPres; ///< Latest long filtered venous pressure (mmHg)
PRESSURE_OCCLUSION_DATA_T;

DD-LEAH-1971-1 28 Jan 2025

Done.

DD-LEAH-1971-1 28 Jan 2025

Done.

DD-LEAH-1971-1 28 Jan 2025

Done.

DD-LEAH-1971-1 28 Jan 2025

Consider putting this code in a static function.

DD-LEAH-1971-1 28 Jan 2025

Why is this commented out? Can we remove it?

DD-LEAH-1971-1 28 Jan 2025

I would change 105 to 9999 or something very high and then add any new s/w faults below these (i.e. 105..).

HD-DENBUG-290-1 29 Jan 2025

RESOLVED in CODE WALKTHROUGH.

HD-DENBUG-290-1 29 Jan 2025

Moved it to the top.

HD-DENBUG-290-1 29 Jan 2025

This is sent only once

LEAHI-DIALIN-LEAH-136-3 05 Feb 2025

I don't see fix.

LEAHI-DIALIN-LEAH-139-3 06 Feb 2025

Fixed. Thanks!

LEAHI-DIALIN-LEAH-136-3 22 Jan 2025

Wrong file name. Not treatment_delivery.py. I would actually prefer something like filtration_purification.py.

HD-DEN-17220-1 07 Feb 2025

Done

LEAHI-DIALIN-LEAH-139-3 11 Feb 2025

Fixed. Thanks!