Keep 2 blank lines above the banner below.

Is this still needed?

You moved this to TxParams.h and I think that's right. I also think all of these other edit treatment parameters message payloads will also need to be moved as well.

Keep 2 blank lines above the test support functions banner.

Not sure why you are making changes to this function.
This function handles message from UI to pause or resume ultrafiltration.
This is not a treatment parameter edit message, so I don't think it's part of your assignment.

No longer needed?

???

Do we need a parameter for confirmation?

We can set bolus volume here now. These functions should be moved to TxParams where all of these settings exist and can be updated.

Why does confirm msg have duration again? If UI resends duration, we have to validate it again. If UI just sends confirmation of previously sent duration, we don't have to validate it again.

These are already in TxParams I believe.

Should these be moved to Services\TxParams?

Change 1 to TRUE

I think these message handlers for editing treatment parameters should be moved to Services\TxParams.

See if we have already defined this stuff in Services\TxParams. TxParams may also have get function(s) for min/max values that you can use.

Added to be consistent. Once institutional records are implemented will store it in there instead of a macro

DO we need bicarb conversion factor as well?

preweight can be non-zero

Add blank line after declarations.

From what I see in UI, it is just a range value not an enum/index

Is it an enum/index like Denali then?

No we are not getting conv factor from UI on it
I will have to confirm with Nick to see if we will get the bicarbConvFactor

Align comments.