I don't have a good option. If we're going to change it to pull approach later, can't we just ignore rate changes when doing fluid bolus with sub fluid until you implement the pull approach? Are we even able to do sub fluid bolus yet? Do we have HDF tube sets?

Sure

I'll implement the pull approach later, this is temporary fix.
We need to run Sub pump at Qb rate, if Qb is updated, how can we increase Qsub?

Should we check to see if blood sitting timer has not exceeded time limit before we resume (similar to what we do from rinseback stop state before we start an additional)?

So we will command BP every 50ms for saline (okay, but not the pull approach we talked about) and send DD Qsub commands every 50ms for sub fluid (probably not a good idea)?

Add blank line between declarations and code.

Fix indenting.

Fix indenting. It's too deep.

Fix indent.

Remove extra blank line.

Fix indent.

Fix indent.

Fix indent.

Give each alarm its own line.

Add doxygen comment to right.

Add blank line between includes and group header.

Library includes should be first and using <> instead of "". Add comment to right explaining what we're including string.h for.

Alphabetize.

Remove if statements. Just call both set functions regardless of current state.

Fix indent. Move up above Dialin "test" message handlers.

Alphabetize.

Follow naming convention that other states have set. This unit should be called StateTxTreatmentEnd.

Shouldn't this be in your TreatmentEnd unit?

Who calls this function? It's public, but no prototype in header file.

What is this function doing in Dialysis unit? And who is calling it? It's public, but I don't see that you added it to header file so nobody can call it anyway.

Why add tx end state here?

Remove extra blank line.

Keep blank line between functions.

Add blank line before return.