align the indent.

Update the naming for 'balChamberSwPeriod' and 'isBalChamberFillinProgress' as per the assignment.

I missed updating this earlier, I’ll push the updated code.

Can't have 3 values. Maximum 2 values allowed.

i see some of the code is for building the code , could you please remove all such code ? or create a new branch and push it , it will confuse. while pushing the code , all supporting code for build / functionality should be removed.

I had this commented again to push to the develop branch. It is uncommented for my testing.

added TODO above

updated as suggested.

updated.

updated accordingly.

added TODO for all

makes sense. removed it.

I think we are using it to send a message to Dialin, to let the tester know whether the command has passed/failed or timed out.

restored

Looks like the description part should only be in the @param part.
sysConfigTreatmentParameters should be an output? Input is None.
Add @details for alarms

align comment

The Id name looks too long? What if we change Temperature to Temp?

Removedd

added

Add @param section for reasons.

Please list names of variables in inputs and outputs.
You can move any description in the brief section.

Please add a comment for this check and break the line in two lines.

Please add a comment about this condition check.

Move this part : ( fabs( request.ufRate - validatedUFRateLhr ) < NEARLY_ZERO ) ) to the next line.

Can we please align (move more towards right) the comments of this newly added section with the comments of existing sections?

Please keep on declaration of the variables in this section and move the initialization to initTreatmentParameters

Please rename this function to initTreatmentParameters

We only add File level or global variables in Inputs and Outputs.
Please remove functions from the input section of this and other functions.