Fixed. Thanks!
RESOLVED IN CODE WALKTHROUGH
Can we do s/w fault now?
Should use of TRo here be for DVT (if V3 s/w config, continue to use TDi)?
This file has been removed.
RESOLVED in CODE WALKTHROUGH.
Remove comment.
Remove blank line.
Align comment.
Not clear this needs to be done. I believe is properly handled in heaters when you get a temperature value and configured for V3.
This function will still override if there any alarms. Are we going to change the order of status report?
Per Sean - "No harm in adding "F" suffix, but not necessary for assignment to a float variable". Will leave as is for float vars.
Removed
Can we just remove this code?
Remove the TRUE case. Just need the else now.
Done.
Looks like blank line is still there.
RESOLVED IN CODE WLAKTHROUGH
Removed the commented-out line.
Add extra blank line here.
Remove extra blank line.
Sensor misspelled (Sesnor). Change name to execFlowSensorMonitor.
Keep blank line.
This block of code is being flagged as a change into my branch but it is not reflected on either commit listed above. Header will be updated in DEN-13801 branch.
Move this to after execHeaters() and before #endif.
Remove
Merge markers need to be removed.
Already removed the merge markers.
Already removed the marge markers.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.
