•  

Comment Results

Review Name Created Custom Fields Content
LEAHI-DD-FIRMWARE-LDT-2936-1 02 Dec 2025

Fixed.

LEAHI-TD-FIRMWARE-LDT-4033-1 27 May 2026

Don't need a flag here. Need a flag in driver and also an isBPCuffConnected() function in driver that we can call from this controller.

LEAHI-TD-FIRMWARE-LDT-4033-1 27 May 2026

Remove.

LEAHI-TESTSUITES-LDT-2174-1 02 Dec 2025

please upload new report to view changes

LEAHI-DD-FIRMWARE-LDT-618-2 04 Dec 2025

You don't need the if statement. The override function will check it automatically.

LEAHI-DD-FIRMWARE-LDT-618-2 04 Dec 2025

Please use getU32override function and then typecast it.

LEAHI-DD-FIRMWARE-LDT-618-2 03 Dec 2025

Should use existing override payload structure (the one with the index) located in TestSupport.h.
Add an enum in this unit for the 5 DD treatment parameters so you can interpret the index field of the message payload to find the specific parameter that is being overridden.

LEAHI-TESTSUITES-LDT-2174-1 03 Dec 2025

The value will be updated based on the target value provided by the user.
To adjust the minimum and maximum values, use the left and right arrows to update them accordingly.

HD-DENBUG-335-1 29 May 2026

This alarm description is more similar to alarm 215, including title, description, trigger conditions?

LEAHI-TD-FIRMWARE-LDT-4027-1 05 Jun 2026

Please add currentPreTreatmentState to inputs.

LEAHI-DD-FIRMWARE-LDT-618-2 04 Dec 2025

Changed the comments

LEAHI-TD-FIRMWARE-LDT-473-1 02 Dec 2025

TDDefs.h is already included in ModePreTreat.h. There is no "TD_Defs.h". Remove this line.

LEAHI-DD-FIRMWARE-LDT-2936-1 03 Dec 2025

publish filter flush status , add other important data , inlet pressure etc

LEAHI-TESTSUITES-LDT-2174-1 04 Dec 2025

Common method is moved to Utility.py file

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

removed

LEAHI-DD-FIRMWARE-LDT-2936-1 04 Dec 2025

Fixed.

LEAHI-TD-FIRMWARE-LDT-473-1 02 Dec 2025

Remove extra blank line.

LEAHI-TD-FIRMWARE-LDT-473-1 02 Dec 2025

I don't see this function resetting Tx params. Probably a good idea to do that, so add call to reset.

LEAHI-TD-FIRMWARE-LDT-473-1 02 Dec 2025

What is this condition here for?

LEAHI-TESTSUITES-LDT-1650-1 03 Dec 2025

can we make this a more general method and put in a shared file and update throughout.. updatet to something like

def setText(obj, text):
    obj["text"] = text
    return obj:
LEAHI-DD-FIRMWARE-LDT-4001-1 11 Jun 2026

fixed thanks

LEAHI-TD-FIRMWARE-LDT-473-1 15 Dec 2025

Thanks for the clarification. This handler already follows the single-parameter MESSAGE_T *message pattern, and the message ID is already defined and registered in Messaging.c. I’ll keep using the existing ID, ensure payload length is validated, and parse the param/value from the payload as expected.

LEAHI-DD-FIRMWARE-LDT-2030-2 09 Dec 2025

Dara Navaei, this is not a CAN message payload, but it does look like a sensor serial message payload. I'm assuming this code works, so sensor does not expect a packed structure?

LEAHI-TD-FIRMWARE-LDT-2240-1 03 Jun 2026

Why is this commented out? We will get stuck in this state if we don't move on to the pressure leak test states.

LEAHI-APPLICATION-LDT-5074-3 29 Jul 2026

removed from settings and config and now handled by fw

LEAHI-TESTSUITES-LDT-2174-1 05 Dec 2025

another mouseClick(waitForObject(confirmButton)) after fw validates parameter is needed

LEAHI-DD-FIRMWARE-LDT-2004-3 08 Dec 2025

These set of valves defined as per system architecture (to make sure all valves initialized as expected).

LEAHI-DD-FIRMWARE-LDT-5209-1 30 Jul 2026

Add comment.

LEAHI-TESTSUITES-LDT-1802-1 08 Dec 2025

Add tst_general_alarm_requirements_instruction to suite.conf file

LEAHI-APPLICATION-LDT-2957-1 10 Dec 2025

moved

LEAHI-APPLICATION-LDT-2957-1 10 Dec 2025

statusColor is so general, but is bound to the heparinStatusColor.
Please either change the status color to be heparin only or change the heparinStatus Color name to be more general.
The latter is better.

LEAHI-TESTSUITES-LDT-2278-1 11 Dec 2025

Renamed the function setObjectText

LEAHI-DD-FIRMWARE-LDT-2185-1 12 Dec 2025

fixed it

LEAHI-TESTSUITES-LDT-2174-1 11 Dec 2025

remove and add delay i waitForObjectExists if needed

LEAHI-DD-FIRMWARE-LDT-2185-1 11 Dec 2025

Add blank line before return statement

LEAHI-DD-FIRMWARE-LDT-2185-1 11 Dec 2025

Need to remove this handler

LEAHI-TD-FIRMWARE-LDT-1886-1 15 Dec 2025

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

LEAHI-TD-FIRMWARE-LDT-1886-1 11 Dec 2025

let public function on top

LEAHI-APPLICATION-LDT-3046-1 18 Dec 2025

updated

TEENSY-BOARD-LDT-2004-6 18 Dec 2025

fixed. thanks!

LEAHI-APPLICATION-LDT-3000-1 18 Dec 2025

updated

LEAHI-DIALIN-LDT-2998-1 18 Dec 2025

Fixed

LEAHI-APPLICATION-LDT-3000-1 17 Dec 2025

keep lock button enabled and update logic

LEAHI-APPLICATION-LDT-3000-1 17 Dec 2025

revert cancel button to older design

LEAHI-APPLICATION-LDT-2825-1 17 Dec 2025

revert

LEAHI-TESTSUITES-LDT-1454-1 16 Dec 2025

the method name we have been using is setObjectText.. please update for consistancy. Test is failing due to this. please test first before committing code

LEAHI-DD-FIRMWARE-LDT-3272-1 27 May 2026

Done

LEAHI-DD-FIRMWARE-LDT-2936-1 02 Dec 2025

Fixed.

LEAHI-DD-FIRMWARE-LDT-2936-1 02 Dec 2025

Fixed.

LEAHI-TD-FIRMWARE-LDT-4033-1 28 May 2026

Fixed