Added but crucible is not showing this change. Please check bitbucket.

These variables are correctly initialized to the first enum in the list.

Not needed because the retry mechanism is going to be deleted in the next ticket.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

Not needed because the retry mechanism is going to be deleted in the next ticket.

Why is this on a separate line? Move after = above.

Remove extra blank line.

Remove extra blank line.

This was added by Vinay to force the chamber F filling.

It was changed by Vinay as part of the Conductivity spikes issue.

Should this be the "Load Syringe" alarm that you just added?

This is a problem, when lower cart pressure, D80 closes and if it is not reached upper cart pressure, it won't open D80

Why do we need 1 minutes of D80 opening ? is it not filling withing 10 seconds ?

bicarbChamberLevel is a local variable and not an input

This is not required , we should design the state machine properly to handle chamber low level

This is not the correct order for state machine,

Can't return in middle of function code.

Remove else and reject command in it and re-test.

At end of transition, do you want to initialize the totalSpentChamberFill reset to zero?

Could you use SpentChamberLevel here, instead calling the getLevelStatus function again? Same for next conditions check as well ( line:889)

Remove extra paranthesis.

Remove empty lines

Mention in comment this is equivalent to Qd:600 minus 50ms switching period to stabilize the valve closure.

Update the comment for Spent chamber fill, max dialysate flow rate (Qd:600) used for fast filling.