Michael Garthwaite

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( ...

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( conductivity sensors ).

tip: use the logger and set the statement to error level and you can see it in runtime

tip: use the logger and set the statement to error level and you can see it in runtime

fixed

fixed

fixed

fixed

fixed.

fixed.

fixed.

fixed.

fixed.

fixed.

fixed.

fixed.

J1 - P9 J2 - P18 J3 - D17 J4 - D27 J5 - D29 J6 - D43 No sensor - D74 (redundant, so we should be able to live without it in short run)

J1 - P9
J2 - P18
J3 - D17
J4 - D27
J5 - D29
J6 - D43
No sensor - D74 (redundant, so we should be able to live without it in short run)

build switch fixes will be completed in the v2 conductivity sensor branch. Per what was agreed on back in November: J1 - P9 J2 - P18 J3 - D17 J4 - D27 J5 - D29 J6 - D43 No sensor - D74 (redundant,...

build switch fixes will be completed in the v2 conductivity sensor branch. Per what was agreed on back in November:

J1 - P9
J2 - P18
J3 - D17
J4 - D27
J5 - D29
J6 - D43
No sensor - D74 (redundant, so we should be able to live without it in short run)

added

added

removed check. the check now happens on before the get.

removed check. the check now happens on before the get.

removed check. the check now happens on before the get.

removed check. the check now happens on before the get.

fixed

fixed

added. DD will now fault

added. DD will now fault

fixed.

fixed.

fixed

fixed

updated.

updated.

Yes since you're not using the return value in execOpModes()

Yes since you're not using the return value in execOpModes()

updated newCondModel to be a U32 before check then casted to enum after. Note: Calculate conductivity has a default statement that will catch any value that is outside of the model enum ( and execu...

updated newCondModel to be a U32 before check then casted to enum after. Note: Calculate conductivity has a default statement that will catch any value that is outside of the model enum ( and execute standard conductivity model )

function removed. DD will now fault if we hit a failed state per execConductivityTeensy

function removed. DD will now fault if we hit a failed state per execConductivityTeensy

how are unsigned integers negative?

how are unsigned integers negative?