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

rejReason = INVALID REQUEST FORMAT

Why would change request from UI come in separate messages (1 for duration and 1 for volume)?

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.

I think these remaining 4 else ifs need to have another && condition that your service is in idle state so that we don't change back into one of these 4 states while a bolus is in progress.

Can we restore the real code here yet? Ask Dara.

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.

Looks like comment for 208 and 223 are swapped?

please align the comments

Please update the msg id to MSG_ID_TD_TREATMENT_SET_POINT_DIALYSATE_TEMP_CHANGE_RESPONSE

Please break it into two lines and check other functions too.

Please break it into two lines