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

0.0F(float type).

I meant 'handleBicarbChamberCartridgeFillWaterEndState function stops the inlet water to cartridge'.

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".

Do we need this?

Change this to work for any state (passed as parameter).

Delete it.

Remove

Alphabetize these includes.

Remove blank spaces after =.

Why is this commented out?

To be deleted this line?

Don't need yet.

Remove all cleaning related functions.

Don't need this yet.

Remove for now.

Missing a comment.

Remove last 5 messages.

Remove these. These are Denali modes.

Add a comment here to wait for BC switching to go Idle state and transition to spent chamber fill state. Also, add a space after (). Apply the same comment in the next Dialysate delivery state as well.

Remove these for now.

I guess we can remove the "balancingChamberFillInProgress" check here.

Remove the TODO comment.

align the comment.

Vendor - TEL - Sameer PoyilWhat's the value you are recommending for?

Remove values in the comment, as they are changing during development (200ms persistence, 4 PSI low pressure, 17 PSI high pressure etc.,), instead use defined persistence, low or high threshold pressure etc.,