•  

Comment Results

Review Name Created Custom Fields Content
LEAHI-DD-FIRMWARE-LDT-2004-1 01 Dec 2025

Why two output lines?

LEAHI-DD-FIRMWARE-LDT-2004-1 21 Nov 2025

Should we be calling get function in monitor instead of driver here?

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

updated. Removed header bar information and device settings information objects

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

Add space before );

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

Update inputs/outputs.

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

Update inputs/outputs throughout this unit.

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

isFlushComplete looks like an output.

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

Update inputs/outputs.

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

Should be 2 blank lines before/after test banner.

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

roRejectionRatioTankFill is also an output.

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

Add blank line between declaration and rest of code in scope.

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

Missing param line in function header.

TEENSY-BOARD-LDT-2004-6 24 Nov 2025

Fix indent.

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

Flag for rejection not needed?

LEAHI-DD-FIRMWARE-LDT-2936-1 26 Nov 2025

comment should start with Capital letter ?

LEAHI-DD-FIRMWARE-LDT-2004-1 01 Dec 2025

All cases seem a little off on indents, especially this one.

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

Modified. Thanks

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

Fixed.

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

This unit is not a mode. It is replacing a mode with a service.
Change to "Treatment parameters unit provides validation and confirmation handling for treatment parameters from the UI."

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

Done in if condition, that block of code will be executed only when the denominator value is greater than 10

LEAHI-TD-FIRMWARE-LDT-3970-1 26 May 2026

please align the comment

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

ive noticed this common method being used in other test cases. If its a general method that is used throughout please move outside in a shared file and call from there. Please update other repositories that use this as well.

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

remove this and use set_parameter_type()

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

We shouldn't delete generated code. Add a #if 0 in this user code section and add a #endif in the user section below the coreEnableFlashEcc() call.

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

By mistake, I forget to remove

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

RESOLVED.

CLOUDSYNC-DENBUG-337-1 28 May 2026

Tiffany Mejia
Which function will create the sentinel sub-folder?

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

Fixed.

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

Remove since we're only publishing when we have new measurements.

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

Checked with Eliza if they need it for alarm persistence time. Not needed at this time, we can add them in future

LEAHI-TD-FIRMWARE-LDT-4013-1 20 May 2026

Should always exec your bolus sm.

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

Yes, I used MSG_ID_AVAILABLE_2.

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

details should have specific lines (e.g. Inputs and Outputs). Generic function description should be in the brief section above.

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

Now using setText(obj, text) method and moved to configuration folder utility.py file

LEAHI-DD-FIRMWARE-LDT-3958-1 29 May 2026

The latest code from staging has an update and fixes this comment. It's not showing here, since the file was already reviewed.

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

Moved this function into shared file

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

Add a doxygen comment above this array.

LEAHI-DD-FIRMWARE-LDT-3958-1 29 May 2026

Added new field for d1 filtered temperature and restored the pressure sensor temperature.

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

Do not mix static and public function, let all public function on top and static bottom, except override and test functions

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

fixed.

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

Added

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

Renamed the function set_Object_Text

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

I don't see any get functions. I think we need to copy from ModeTxParams.c.
Let's make sure we've copied all needed functions from ModeTxParams.c and then don't forget to delete ModeTxParams.c/.h.

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

Put recirc state after Rx state (both here in switch statement and also in enumeration).

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

Change comment from 3 to 5 minutes to match code.

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

Please add the test case to the suite.conf

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

add comment block to all functions that are missing

LEAHI-TD-FIRMWARE-LDT-4072-2 27 May 2026

device and register is fine. Or deviceSel and registerSel.

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

yes good catch. updated

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

Put in alphabetical order (before Timers.h).