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.

Even if you do not have an Output, please add the tag with None as the value.
Please fix in other functions as well.

Please indent the doxygen comments of all the functions, similar to existing functions.

Updated

origTreatmentParams should be an output?

Please enclose each condition in parenthesis

Can you please move the TODO comment above this line?

Please add a comment for this check.

Please add @param

Please add @param

treatParamsConfirmed should be an output?

Delete extra line

Please assign it to FALSE

Please add details of you message Id by using this format.

@details \b Message \b Sent:

Place add \b before Outputs.

Place add \b before Outputs.

Why did we remove this? This is needed for BC Switch only condition.