Conductivity.c

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 8 more files in changeset.
right now pre and post are same. make a TODO to update post later

right now pre and post are same. make a TODO to update post later

its mixing control interval

its mixing control interval

could you document th math behind the calculation also

could you document th math behind the calculation also

LDT-4734: code review comments

  1. … 2 more files in changeset.
What is 4 (or 2)? Should be #define or enum or const.

What is 4 (or 2)? Should be #define or enum or const.

please align this comments with below structure

please align this comments with below structure

Please align the comments

Please align the comments

could you please align it ?

could you please align it ?

Add blank line (should be 2 before test support banner).

Add blank line (should be 2 before test support banner).

Remove blank line.

Remove blank line.

Needs doxygen comments.

Needs doxygen comments.

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

fixed. thanks!

Reverted change. Added uncompensated conductivity to the conductivity publishing

Reverted change. Added uncompensated conductivity to the conductivity publishing

LDT-4734: code review comments

  1. … 2 more files in changeset.
Revert "LDT-4734: updates conductivity resistance messages."

This reverts commit 14277e7c55fbb8407612572aca191ffae0835796.

  1. … 1 more file in changeset.
space after if.

space after if.

Space after if.

Space after if.

If it is useful for current and future conductivity sensor development, you may keep it with TODO comment to remove later once testing is done.

If it is useful for current and future conductivity sensor development, you may keep it with TODO comment to remove later once testing is done.

so, are we removing this? I think it is better to have this as this might make it easier to get and validate the values in future. Not sure what others think.

so, are we removing this? I think it is better to have this as this might make it easier to get and validate the values in future. Not sure what others think.

Remove empty line.

Remove empty line.

Align the indentation

Align the indentation

Align the indentation.

Align the indentation.