Maintain alphabetical order.

Eliza says we do allow multiple iso UF sessions in a treatment. So do we still need this else if, but looking at <= iso UF elapsed session time?

What about HDF?

Once UI software sends the volume and/or duration change request and once we change them, the UI software will send another request to confirm the displayed settings to the user and that is what is being handled in this function.

return result;

Keep local result and set to TRUE or FALSE with a ternary.

Which setting does this message confirm (duration or volume)?

Align comment.

· /firmware/App/Modes/ModePreTreat.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Modes/ModeTreatment.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Monitors/Pressures.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Monitors/Pressures.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/Messaging.h	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.c	(-2, +2)	| History	| Source	| Diff
· /firmware/App/Services/TxParams.h	(-2, +2)	| History	| Source	| Diff

]]> Fri, 29 May 2026 14:57:06 -0700 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3970-1 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3970-1 Sean Nash and Vendor - TEL - Arpita Srivastava finished reviewing LEAHI-TD-FIRMWARE-LDT-3970-1 LEAHI-TD-FIRMWARE-LDT-3970_Edit Treatment Parameters During Treatment - FW Implementation - 1/4: ]]> Fri, 29 May 2026 14:33:51 -0700 Vendor - TEL - Arpita Srivastava https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3970-1/#c29528 https://devapps.diality.us/cru/LEAHI-TD-FIRMWARE-LDT-3970-1/#c29528 Vendor - TEL - Arpita Srivastava commented on LEAHI-TD-FIRMWARE-LDT-3970-1

Please add the min and max for tmp as well in the outputs.

How can settings be pending when you set that flag to TRUE in the code below this? And why would we want to use uf volume setting in the else part of this ternary?

else reason is Invalid Request Format

Should check if iso UF volume + UF volume < max. And if user can do multiple iso uf sessions, you would check iso UF volume + any previous iso UF volume + UF volume.
I don't know if there is a minimum iso UF volume.

Looking for "less than current" only makes sense if user can do multiple iso UF sessions in a treatment. And if that's the case, we should be looking at how much time we've already spent doing iso UF instead of how much time we've spent in the HD or HDF treatment.

I think we have to check whether the iso uf duration + treatment duration does not exceed maximum treatment duration.
And I'm not sure if there is a minimum duration for iso uf, but if there is a minimum, it probably isn't the same as the minimum treatment duration.

Just add this as an AND condition in the if statement above.

If we are in treatment end state, the treatment is completed (no need to check that) and I don't think iso UF is a modality.

Do we need this? Check with Raghu. I don't think iso UF is a modality (at least not yet).

Align all or none of the =

I don't think iso uf is considered a modality - at least not in phase 1 of development.

Use the NEARLY_ZERO constant (instead 0f 0.01F) like the other one.

Add a doxygen comment above this array.

Add a doxygen comment above this array (similar to structures).

Add a doxygen comment above structure for the whole structure (see structure above for example).

Add a doxygen comment for this definition.

remove camel case for SetPoint, instead use Setpoint
Update line 659 too

Can we rename this to TREATMENT_DURATION_CONFIRM_REQUEST_PAYLOAD_T

Can we rename this to TREATMENT_DURATION_VALIDATE_REQUEST_PAYLOAD_T?

What is the difference between the bicarbonateConcentrate and bicarbonate. Please update the comment.