I agree but it is already in the init function.

Swap order of == for these 2 if statements - put constant on left side.

Why?

Why is UF volume being added here? If we add it, we need to change DD f/w too. But it is not clear to me why DD f/w needs to know the UF volume.

Use ternary operator to explicitly set to TRUE or FALSE.

Our TODO comments are all CAPs.

I think these if statements should be looking for valve state == VALVE_STATE_IDLE, not <

Fix implementation to return a BOOL (TRUE if SW_COMPATIBILITY_REV == ddVersion.compatibilityRev.

Should return T/F. Change comment to "TRUE if DD f/w is compatible with TD f/w, FALSE if not.".

Should return a BOOL (that is what ModeStandby is expecting).

Fix isDDCompatible() function so that it returns a T/F instead of a compatibility revision.

Align comment.

Why are these added? And why commented out. H13 and H20 are not rotary valves - they are 3 way valves. They do not belong here.

Should we handle blood sitting timer before publish? Are we publishing anything re: blood sitting?

Add TODO to these comments.

I think ( setTxDur - preStop ) should be changed to ( preStop ).
Also, I think preStop should be renamed to hepDurMin.

Change INPROGRESS to OPEN (because you're going to remove INPROGRESS).

You can do this sendMessage just once after else.

Should this be the "Load Syringe" alarm that you just added?
Also, does this alarm show the user how much heparin to load into the syringe? I wasn't expecting this to be an alarm.

Can we remote this TODO comment?

Remove blank line.

We should go to dry self test state. Why did this change to Rx state?

Comment - should be "install" not "instll".

· /firmware/App/Modes/ModeTreatment.c	(-6, +11)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.h	(-3, +5)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxIsolatedUF.c	(-4, +129)	| History	| Source	| Diff
· /firmware/App/Modes/StateTxIsolatedUF.h	(-0, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-0, +1)	| History	| Source	| Diff

]]> Tue, 21 Jul 2026 13:19:53 -0700 Vendor - TEL - Vijay Pamula https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4028-2 Vendor - TEL - Vijay Pamula Closed LEAHI-TD-FIRMWARE-LDT-4028-2 ]]> Tue, 21 Jul 2026 10:35:53 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4063-1/#c30582 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-4063-1/#c30582 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-4063-1

add a space before and after 0

add a blank line above

add a blank line above

minor spell issue : change responceStatus to responseStatus

add offRequestAwaitingUserConfirmation and offRequestPendingTimer to outputs