Yes, this needs to be corrected. I don't see any controller existing properties defining these min and max. Will discuss further with you on this.

Since feedforward output is same as control signal output, checking control signal min and max seems to be correct.

Reduced gain and testing in progress.

Reduced gain and testing in progress.

Done.

Done.

its auto generated . should not change it manually. any way when merging its added correct value

Now we don't need this macro ?

now we are initializing sensor simultaneously?

comments missing

comments missing

This is not clear- please give a detailed description for this variable

use capital for starting letter

please align the comments

please Alignment the comments

Why the order is important here ? please document it

do we need bracket here ? all text should start with capital

Why a * here ?

is it write states or read state ?

is documentation correct ?

Why do some functions take sensor as a U32 and some as a CONDUCTIVITY_SENSORS_T? The latter seems appropriate.

Remove blank line.

Remove blank line.

Add blank line before return.

Add blank line before return.

Remove blank line.

Not sure these min/max are right here either.

So we have all temp sensors getting averaged now, right? So should we have one get function that takes a sensor ID as param?

Remove extra blank lines.