I think this check is for over-fill. Counter is for illegal level sensor readings. Over-fill check should have it's own counter (or use new persistence style) and we should reset that separate counter in the "else" condition.
Fixed. Thanks!
Remove extra blank line.
Remove blank line.
RESOLVED in CODE WALKTHROUGH.
Move up to line ~108 where the other shift #defines are.
Remove references to this flag.
Needs doxygen comment.
Comment doesn't match time.
Remove TODO comment.
Remove TRUE case. Just need the else now.
Can we remove this code?
Done.
RESOLVED IN CODE WALKTHROUGH.
Should initialization use 0.0F?
RESOLVED IN CODE WALKTHROUGH
Keep blank line.
Remove comment at end of line.
No explanation for commenting this function.
Merge markers need to be removed.
conversion.py
We can have either of the values set without the other value set.
Should 365 be #define? Why changed from #define to static const U32?
Fixed.
Removed all blank lines.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.
