•  

Comment Results

Review Name Created Custom Fields Content
LEAHI-TD-FIRMWARE-LDT-3126-1 15 Jan 2026

Remove reset override functions. Reset should be handled in the single (general) override command handler.

LEAHI-APPLICATION-LDT-2805-1 06 Jan 2026

do you need to also register the following that are imported in main.qml?

VTreatmentAdjustmentBolusVolume
VTreatmentAdjustmentDurationEdit
VTreatmentAdjustmentDurationConfirm
VTreatmentAdjustmentSetPoints
VTreatmentAdjustmentBloodFlowRate
VTreatmentAdjustmentDialysateFlowRate
VTreatmentAdjustmentDialysateTemperature

LEAHI-APPLICATION-LDT-2805-1 06 Jan 2026

Properties are being set to zero by default.

LEAHI-APPLICATION-LDT-2805-1 06 Jan 2026

Updated.

LEAHI-DD-FIRMWARE-LDT-4183-1 01 Jun 2026

For all floats add F at the end, across the file.

LEAHI-TESTSUITES-LDT-1454-1 07 Jan 2026

changed this Ultrafiltration in one line

LEAHI-TD-FIRMWARE-LDT-4009-1 01 Jun 2026

What about HDF?

LEAHI-DIALIN-LDT-4485-1 09 Jun 2026

Add blank line as required.

LEAHI-DD-FIRMWARE-LDT-2030-2 07 Jan 2026

done

LEAHI-DD-FIRMWARE-LDT-2030-2 07 Jan 2026

followed the pattern.

LEAHI-DD-FIRMWARE-LDT-2030-2 07 Jan 2026

If there are no messages from the BloodLeak.h then we should remove the include.

LEAHI-TESTSUITES-LDT-3588-1 11 Jun 2026

Obsolete branch to be deleted. Closing reivew

LEAHI-DIALIN-LDT-2998-1 08 Jan 2026

Fixed

LEAHI-TESTSUITES-LDT-1454-1 09 Jan 2026

Updated the test scripts as per the suggestion

LEAHI-APPLICATION-LDT-3139-1 09 Jan 2026

added comment

LEAHI-DD-FIRMWARE-LDT-2185-1 09 Jan 2026

fixed it

LEAHI-TD-FIRMWARE-LDT-3126-1 09 Jan 2026

Added

LEAHI-DD-FIRMWARE-LDT-2030-2 07 Jan 2026

Remove extra blank line.

LEAHI-DD-FIRMWARE-LDT-2030-2 07 Jan 2026

Add space between (( and )).

LEAHI-DD-FIRMWARE-LDT-2185-1 13 Jan 2026

Fixed

LEAHI-TESTSUITES-LDT-1454-1 06 Jan 2026

spelling error should be "title"

LEAHI-DD-FIRMWARE-LDT-3159-1 15 Jan 2026

Fixed

LEAHI-DD-FIRMWARE-LDT-4001-1 09 Jun 2026

fixed. thanks!

LEAHI-DD-FIRMWARE-LDT-4001-1 09 Jun 2026

added.

LEAHI-TESTSUITES-LDT-2174-1 16 Jan 2026

Updated the value to all combo boxes

LEAHI-DD-FIRMWARE-LDT-4001-1 09 Jun 2026

removed

LEAHI-TESTSUITES-LDT-2174-1 16 Jan 2026

Updated the value to 120

LEAHI-TD-FIRMWARE-LDT-3126-1 15 Jan 2026

Only need 2 message IDs here. In Leahi, we don't have separate messages for set vs. reset. Both set and reset are handled in the same message.

LEAHI-TD-FIRMWARE-LDT-3126-1 15 Jan 2026

Remove "Set" from override function names. These 2 override functions should handle both set and reset (from msg payload).

LEAHI-DD-FIRMWARE-LDT-4011-1 12 Jun 2026

Remove list of states from comment. It is not a complete list and we shouldn't even try to list them here anyway.

LEAHI-APPLICATION-LDT-3106-1 22 Jan 2026

Updated.

LEAHI-DIALIN-LDT-4511-1 02 Jun 2026

If you are overriding MSG_ID that's in use, please move it to it's new location instead of fully removing to avoid breaking the SW.

MSG_ID_TD_SYRINGE_PUMP_DATA = 0xB1

LEAHI-TD-FIRMWARE-LDT-4063-1 09 Jun 2026

From here forward, we can just call sendMessage() with appropriate params.

LEAHI-TD-FIRMWARE-LDT-4033-1 11 Jun 2026

Initialize bpResults.

LEAHI-DD-FIRMWARE-LDT-4011-2 08 Jun 2026

do we have to leave a blank line after this?

LEAHI-DD-FIRMWARE-LDT-4011-1 12 Jun 2026

Is this normal UF volume or normal UF volume + Iso UF Volume?

LEAHI-APPLICATION-LDT-2805-1 06 Jan 2026

View still exists, instance has been moved to main.qml

LEAHI-DD-FIRMWARE-LDT-2030-2 07 Jan 2026

changed.

LEAHI-TD-FIRMWARE-LDT-4009-1 01 Jun 2026

Which setting does this message confirm (duration or volume)?

LEAHI-APPLICATION-LDT-2805-1 08 Jan 2026

Updated.

LEAHI-DD-FIRMWARE-LDT-2185-1 09 Jan 2026

fixed it

LEAHI-DD-FIRMWARE-LDT-2185-1 13 Jan 2026

Updated

LEAHI-APPLICATION-LDT-4561-1 09 Jun 2026

ask sean

LEAHI-TD-FIRMWARE-LDT-4063-1 09 Jun 2026

Remove change.

LEAHI-TD-FIRMWARE-LDT-4033-1 11 Jun 2026

setNIBPCommand is a function, not a variable. I think Outputs should be "none"

LEAHI-DIALIN-LDT-4001-1 02 Jun 2026

I would like to ask to check out the substitution pump on the develop branch and update it based on that.
There are some changes thanks to the merging of the refactor.

LEAHI-DD-FIRMWARE-LDT-4001-1 09 Jun 2026

renamed to reflect whats on staging

LEAHI-DD-FIRMWARE-LDT-2185-1 14 Jan 2026

M3 or P10 ? need a good documentation

LEAHI-DD-FIRMWARE-LDT-2185-1 14 Jan 2026

sensor name required in comments

LEAHI-DD-FIRMWARE-LDT-2185-1 14 Jan 2026

If this function is only called from here, should we just move the function's code into this function (i.e. combine the functions)?