•  

Comment Results

Review Name Created Custom Fields Content
LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

remove duplicate import

LEAHI-TESTSUITES-LDT-1218-1 15 Oct 2025

RESOLVED

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Remove this import as its not being used

LEAHI-TD-FIRMWARE-LDT-473-3 06 Apr 2026

Can we just remove this?

LEAHI-DD-FIRMWARE-LDT-1195-2 06 Apr 2026

Remove blank line

LEAHI-DD-FIRMWARE-LDT-4733-1 09 Jul 2026

fixed. thanks!

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

update all temp -> tmp and Temperature - > Trancememberance

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Remove this import as its not being used

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

in_range_color and out_of_range_color are not being used so please remove

LEAHI-DD-FIRMWARE-LDT-4614-1 08 Jul 2026

TODO?

LEAHI-TD-FIRMWARE-LDT-4029-1 09 Jul 2026

I believe the SyringePump controller will trigger this alarm at end of seek operation.

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

align indentations

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

align indentations

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Please remove all the imports not being used

LEAHI-TESTSUITES-LDT-1218-1 15 Oct 2025

you can remove the top portion as its not correct. This will get generated later

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

remove commented out line

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

venous_RangeBar

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

dial_tmp_value

LEAHI-TESTSUITES-LDT-1218-1 14 Oct 2025

when adding new items please organize and align as done initially

LEAHI-TESTSUITES-LDT-1057-1 14 Oct 2025

if this is added by us please add and push in the test suite. My test suite can not run

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Removed the pressure_text_obj method.

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Removed visible parameter and aligned all objects.

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

align indentations

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Please remove all the waits. It is not needed and just makes the test run longer. The quish.waitForObject should be your wait by giving it a timeout. If the object can not be found then add a longer timeout.

e.g.
arterial_rangeBar = utility.get_object_from_names(names.arterial_RangeBar, error_message="arterial rangeBar object is missing", timeout_ms=1000)

LEAHI-DIALIN-LDT-2352-1 16 Oct 2025

done - as part of this comment, the piston pump module was deleted

LEAHI-DD-FIRMWARE-LDT-2004-9 18 Mar 2026

Add blank line after declarations.

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Remove the following as I do not see these being used anywhere

mainTreatmentScreen_arterialRangeBar_RangeBar
mainTreatmentScreen_arterialRangeBar_RangeBar_marker
mainTreatmentScreen_venousRangeBar_RangeBar
mainTreatmentScreen_venousRangeBar_RangeBar_marker
mainTreatmentScreen_tempRangeBar_RangeBar
mainTreatmentScreen_tempRangeBar_RangeBar_marker

LEAHI-DIALIN-LDT-2428-1 16 Oct 2025

how come we need to add the path here?

TD-LDT-1886-7 16 Oct 2025

Replace measured current with measured torque.

LEAHI-TESTSUITES-LDT-1218-1 16 Oct 2025

Removed all waits.

LEAHI-TESTSUITES-LDT-1218-1 15 Oct 2025

Remove visible parameter and please align spacing between each how it was before along with the new objects

LEAHI-DD-FIRMWARE-LDT-3103-1 12 Mar 2026

fixed.

LEAHI-DD-FIRMWARE-LDT-3103-1 12 Mar 2026

Remove blank line.

LEAHI-DD-FIRMWARE-LDT-3103-1 12 Mar 2026

I see we are doing this same logic in multiple places. Can we just have a function that returns the minimum speed for the given dialysate pump?

LEAHI-DD-FIRMWARE-LDT-3352-1 12 Mar 2026

Done.

LEAHI-DD-FIRMWARE-LDT-3603-1 11 Mar 2026

if ( TRUE == testChangeSubMode )

LEAHI-DD-FIRMWARE-LDT-3103-1 12 Mar 2026

declared a fill timeout alarm

LEAHI-DD-FIRMWARE-LDT-3103-1 13 Mar 2026

Looks the fields are empty and misaligned. fill as needed.

LEAHI-APPLICATION-LDT-3413-1 13 Mar 2026

BN APPROVES.

LEAHI-APPLICATION-LDT-3516-1 13 Mar 2026

BN APPROVES.

LEAHI-DD-FIRMWARE-LDT-3352-1 13 Mar 2026

Remove extra blank line.

LEAHI-TD-FIRMWARE-LDT-473-3 06 Apr 2026

Align comments.

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

updatead

LEAHI-DD-FIRMWARE-LDT-3352-1 13 Apr 2026

Yes, this needs to be corrected. I don't see any controller existing properties defining these min and max. Will discuss further with you on this.

LEAHI-DD-FIRMWARE-LDT-2008-1 14 Apr 2026

fixed

LEAHI-DD-FIRMWARE-LDT-2008-1 14 Apr 2026

fixed

LEAHI-DD-FIRMWARE-LDT-2008-1 10 Apr 2026

comments missing

LEAHI-DD-FIRMWARE-LDT-2008-1 10 Apr 2026

please align the comments

LEAHI-DD-FIRMWARE-LDT-2008-1 10 Apr 2026

use capital for starting letter

LEAHI-DD-FIRMWARE-LDT-3777-1 14 Apr 2026

fixed.