updated now

Updated

Updated

Updated

Updated

Updated

Updated

We are checking level status 3 times in same function. So Vinay suggested to use local variable to avoid the inconsistent states across the function.

keep a blank line between declaration(s) and code.

If level can be overridden, you should use get function.

Add blank line before banner.

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.

Delete it.

Remove blank spaces after =.

To be deleted this line?

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.

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

Add TODO to comment.