Lines of code count not available
Use ternary to explicitly assign TRUE or FALSE.
Done.
I think we also need this to be uncommented, as we were running into an alarm because of this.
Added new field for d1 filtered temperature and restored the pressure sensor temperature.
The latest code from staging has an update and fixes this comment. It's not showing here, since the file was already reviewed.
as discussed, these were outcome of the test results. we will review these gains with latest controller.
restored
Not sure why Crucible is showing it this way. Let's just re-review these diffs.
Should be #ifndef
Should be #ifndef I think.
Can we make these low priority and 999 rank?
It is weird that we're responding so far from request. It is not clear to me that Dialin has requested anything at this point.
Details looks weird. Where is inputs and outputs like other function headers?
Add a blank line after banner.
Please restore before you merge to staging.
If I'm reading this right, we're saying that if bad CRC alarm is NOT enabled, initialize the alarm? That seems backward.
Same comments as ENABLE_ALARM_1 above.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.