My understanding is that Standby should stop all pumps and heaters, including D108. Although the D108 driver implementation is not available yet, I thought keeping this call would preserve the intended Standby behavior and allow it to work once driver support is added. Could you please let me know if there is any concern with leaving it in place for now?

Michael confirmed that FP will eventually support two firmware-controlled sampling modes: Filter Sample and RO Permeate Sample. Since the flow paths are not defined yet, we will keep a generic placeholder for the sampling mode/state and update the implementation and requirements once the feature is better defined.

Add TODO or revert back changes.

Delete empty line

Add the comment.

Align the indent throughout.

Add comment.

Space before and after paranthesis.

If debug is complete, do you want to revert back to actual values? if you still need it for some more time, please add TODO comment.

Update the comment.

Aling the indentation of value field.

Align the indentation.

updated now

Add the header description.

Do we need to transition to next state DD_PREGEN_DRY_BICART_INITIAL_FILL here?

Remove empty lines.

Add comments across the file.

Add comment.

Add a test support for Pregen fill request.

Update header description for all newly added functions.

You may need to check other flags such as DryBicartFillRequested( Initial Fill), bicarbChamberFillRequested (Filling Chamber F), instead of 'bicarbChamberPregenFillRequested'. Keep Drain and DepressurizeRequest flag as is. This makes sure at any given point, dryBicartExec calls one of the service only.

Same here adding 'bicarbChamberPregenFillRequested' flag.

Same here adding 'bicarbChamberPregenFillRequested' flag.

Same here adding 'bicarbChamberPregenFillRequested' flag.

Retain the previous( dryBicartFillRequested) and add 'bicarbChamberPregenFillRequested' flag here.

PregenFillExecState? may be creating a new fault ID for pregen fill.

Update description as required.