•  

Comment Results

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

Renamed the test case as tst_main_treatment

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

Aligned.

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

Remove if not being used

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

Please align all as following:

        { "arterial_pressure"    : 100,  "arterial_limit_low" : -300, "arterial_limit_high" : 120, 
          "venous_pressure"      : 140,  "venous_limit_low"   : -100, "venous_limit_high"   : 250,
          "pressure_limit_state" : 2,    "tmp_pressure"       : 250 , "tmp_limit_low"       : -400 , "tmp_limit_high" :  200 },
LEAHI-DD-FIRMWARE-LDT-3630-1 12 Mar 2026

I think the 2 initial rpm constants are inputs. The test config we would normally not include.

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

dialyste_tmp_title_Text

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

Please remove o_Overlay and o_treatmentHome as they are not used

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

If this is specific to only your test case then move this in the test case

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

source removed

LEAHI-TD-FIRMWARE-LDT-3126-2 16 Mar 2026

Match properties from Denali where you can. The priorities should both be HIGH. The rank for the warning should be a higher number. The alarm should be TRUE for no resume and no rinseback. So, I know these properties aren't correct.
One thing I don't think we should copy from Denali though - I think the warning should not be TRUE for the STOP property so that we stay in blood prime state.

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

Update the comments - Valve 4, D21

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

This message ID (0xF0) is no longer valid? and remove empty spaces.

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

align "unamed" to others

LEAHI-TD-FIRMWARE-LDT-3126-2 13 Mar 2026

Assign bloodPrimeStartMS here to mark start of pause.

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

Removed all waitForGUI commands.

LEAHI-APPLICATION-LDT-3698-1 16 Mar 2026

Add '#' to start of line?

LEAHI-TD-FIRMWARE-LDT-1903-9 11 Mar 2026

You are calling setAirPumpState in if and else. Consider just calling once after the if/else.

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

Why (void)?

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

after some thought, test_training.py will need to know where leahi dialin is if leahi_dialin is not installed in the venv. This is okay

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

remove all the waitForGUI

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

the variable name is not "temp" it is "tmp" for Trancememberance. Please fix all object names that are "temp" and update to "tmp" uppdate comments from "temperature" to "Trancememberance"

TD-LDT-1886-7 16 Oct 2025

BloodFlow.c should have a public get function for torque like other overrides.

LEAHI-RO-FIRMWARE-LDT-1773-1 16 Oct 2025

I think this should be initialized to zero in case some bits are unused.

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

I think we should replace motor current with torque instead of adding torque as new field.

LEAHI-DD-FIRMWARE-LDT-3749-2 10 Apr 2026

You are right, modified in all places

LEAHI-DD-FIRMWARE-LDT-2004-9 01 Apr 2026

Align comment.

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

now we are initializing sensor simultaneously?

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

This is not clear- please give a detailed description for this variable

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

do we need bracket here ? all text should start with capital

LEAHI-TD-FIRMWARE-LDT-4040-1 14 Jul 2026

Update function header format from Denali to Leahi style (e.g. \b for inputs/outputs) throughout this file.

LEAHI-DD-FIRMWARE-LDT-3735-1 14 Jul 2026

Remove blank line.

LEAHI-TD-FIRMWARE-LDT-4040-1 10 Jul 2026

What if multiple confirmations pending? Which ID will this function return?

LEAHI-TD-FIRMWARE-LDT-4040-1 14 Jul 2026

Update all function headers in this file to Leahi style (e.g. \b before inputs/outputs).

LEAHI-DD-FIRMWARE-LDT-3735-1 14 Jul 2026

Add blank line between declarations and rest of function code.

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

done

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

Update the comments = Valve 7, D26.

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

rename test case to tst_main_treatment.. The name looks like this is only testing pressure when we test all the other items as well

LEAHI-DIALIN-LDT-3824-1 08 Apr 2026

Fix the header

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

Changed and left them as static, as we don't need these outside this unit

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

Done.

LEAHI-DIALIN-LDT-3629-1 12 Mar 2026

added Todo comment

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

Can we delete this commented out line?

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

currentMode is an input.

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

if ( TRUE == result )

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

is it write states or read state ?

LEAHI-APPLICATION-LDT-3574-1 12 Mar 2026

update to c++ method

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

updated sampling to be determined on read count rather than fixed timer.

LEAHI-DD-FIRMWARE-LDT-2004-9 01 Apr 2026

Add blank line after declarations.

LEAHI-TD-FIRMWARE-LDT-3126-2 16 Mar 2026

Aligned

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

added