•  

Comment Results

Review Name Created Custom Fields Content
TD-LDT-1886-7 10 Nov 2025

documentation error- is it transition start time ?

LEAHI-TD-FIRMWARE-LDT-1886-2 05 May 2026

How are we entering the complete state? If we are planning to implement this function later, can we please add a TODO?

LEAHI-TD-FIRMWARE-LDT-1886-2 05 May 2026

The public function name signalGoToInstallState is not sufficiently descriptive. Would renaming it to requestTubeSetInstall() be appropriate?

HD-DENBUG-328-1 12 Nov 2025

RESOLVED IN CODEWALKTHROUGH

LEAHI-DD-FIRMWARE-LDT-3980-1 28 Apr 2026

This doesn't seem to be correct. Persistence should start once fill complete pressure is achieved, not after D65 opening?

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

Already set this above.

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

updated

LEAHI-TD-FIRMWARE-LDT-2240-1 07 May 2026

yes, removed

LEAHI-DD-FIRMWARE-LDT-2662-1 12 Nov 2025

updated to now change result to FALSE if we get a rr that is not NONE

LEAHI-APPLICATION-LDT-3932-1 05 May 2026

remove demo code

LEAHI-TESTSUITES-LDT-2951-1 08 May 2026

Changed TD Opmode from Standy state to Treatment state

LEAHI-DD-FIRMWARE-LDT-3963-1 05 May 2026

Remove this flag.

LEAHI-APPLICATION-LDT-2727-1 13 Nov 2025

this method removed but updated elsewhere

LEAHI-APPLICATION-LDT-2727-1 13 Nov 2025

updated

LEAHI-TESTSUITES-LDT-1078-1 11 Nov 2025

remove this file and port anything needed into config.py for consistency just like we are doing in http://devapps.diality.us:8060/cru/LEAHI-TESTSUITES-LDT-1650-1

LEAHI-DD-FIRMWARE-LDT-2004-2 12 Nov 2025

Move to ConductivityTeensy.c?

LEAHI-DD-FIRMWARE-LDT-2004-2 12 Nov 2025

#ifdef

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

Why exposed?
Shouldn't the SettingsBase decide where the children must anchor, or locate?

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

For signals and slots as we tlaked use blocks of

onSomething: {
   aFunctionCall()
}


Unless there are multiple similar calls that need alignment in one line.

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

Please make vDateTime optional; if not set, return the current datetime epoch.
It is better to use the 'timeEvent' (right now, onSetDateUIFinished).

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

Align '_', and '=' please.

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

Same as https://devapps.diality.us/cru/#LEAHI-APPLICATION-LDT-2727-1CFR-76783

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

Add the comments for coefficients. bring a top-level comment explaining the formula where these coefficients are being used.

LEAHI-DIALIN-LDT-3505-1 07 May 2026

Please remove for the above reasons.

LEAHI-TESTSUITES-LDT-3208-1 28 Apr 2026

I thought we made the 3000 timeout the default value and need to be set unless it is different.
Please correct me if that is fixed on another branch.

LEAHI-TD-FIRMWARE-LDT-4149-1 27 Jul 2026

What is happening here? Not clear if we need cartridge param. Let's remove it for now.

LEAHI-DD-FIRMWARE-LDT-4410-3 27 Jul 2026

assigned 0.0F as per sameer's suggestion

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

Added

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

Add rinsebackPublishTimerCtr to outputs

TD-LDT-1886-7 08 Nov 2025

The checks are back in a generic function.

LEAHI-DD-FIRMWARE-LDT-2004-2 11 Nov 2025

use uniform naming convention. All other places used conductivity

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

This should be FS (file system), not the OS, unless you run a script and it is being executed by bash/sh.

LEAHI-APPLICATION-LDT-4193-1 07 May 2026

Separate models in separate classes headerbar and water sample

LEAHI-DD-FIRMWARE-LDT-4188-1 30 Apr 2026

To resolve the build failure with DD, haven't added all the features, instead added TODO comment

LEAHI-DD-FIRMWARE-LDT-4183-1 11 May 2026

Is this defined elsewhere? We shouldn't define this in multiple places. I think it should be in one header file - maybe balancing chamber controller.

LEAHI-DD-FIRMWARE-LDT-2662-1 12 Nov 2025

reworked to use the request functions we have in the respective op modes. They check the standby constraint

LEAHI-APPLICATION-LDT-2727-1 12 Nov 2025

Why do we need to remove this?

LEAHI-DD-FIRMWARE-LDT-2004-2 12 Nov 2025

#ifdef USE_TEENSY_CONDUCTIVITY_DRIVER

LEAHI-TESTSUITES-LDT-3208-1 28 Apr 2026

please align imports.

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

Remove extra line.

LEAHI-TD-FIRMWARE-LDT-1886-2 05 May 2026

add a space between message and )

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

remove blank line.

LEAHI-TESTSUITES-LDT-1650-1 12 Nov 2025

I have deleted strings.py file from Bitbucket branch feature/LDT-1650-headerbar-information

LEAHI-APPLICATION-LDT-3932-1 05 May 2026

change to closeOnStateChange

LEAHI-APPLICATION-LDT-2727-1 13 Nov 2025

updated

LEAHI-APPLICATION-LDT-2727-1 13 Nov 2025

this was existing code.. Updated to use the DeviceController and made updates accordingly. And Tested and still works

LEAHI-APPLICATION-LDT-4161-1 07 May 2026

BN Appprvoes

LEAHI-SW-SCRIPTS-LDT-2727-1 13 Nov 2025

Updated to follow format

LEAHI-APPLICATION-LDT-4161-1 07 May 2026

BN Appprvoes

LEAHI-SW-SCRIPTS-LDT-2727-1 12 Nov 2025

Please use the following commands from NetworkManager utilities, since in Leahi we are not using the wpa_supplicant directly:

nmcli connection delete DilaityIoT 
nmcli connection delete $(nmcli -t -f NAME connection show --active)