why TODO?

Do we need to close D64 as well?

Delete the commented line

handleBicarbChamberInitiateSupply?

Update this.

Remove this comment.

Update Inputs and Outputs throughout the file.

Do we need this?

when are we closing D64 valve? handle the same in else block as well.

Can this be assigned during declaration time?

are we using these?

Can we use one macro (instead of three) for the D66 persistence?

The difference between the actual pressure measured at bicart assembly and the measured D66 pressure based on the placement of the sensor.

Dry bicart data publish in the comment.

fix the typo in 'persistence' across the file.

This function is not checking the level anymore and may be move the functionality at end of fill water end state and transition to next chamber fill state directly,

Check pressure limit.

Does this supposed to be 14 or 17 PSI (based on manifold) for B2.1 hardware?

Is this code updated for B2.1 fill implementation?

Update Inputs and outputs.

fix the typo.

declare a local variable to get these function values and use the variables here?

remove blank space before =

Can we split the functions into two files: 1) DryBicart.c/h - Manages drybicart fill, supply, drain & depressurizing states and 2) MixingControl: for acid/bicarb open/close loop functionality?

Same as above.

Can we keep getBicarbMixControlInterval( removing drybicart) from the function name?

Do we need to call updateDryBicartOperation() here to determine the appropriate state.

remove blank lines

Align comment

function adjusts the volume of acid and bicarb based on the target conductivity of bicarb and dialysate solution?