Lines of code count not available
This dead band control ? here is the ideal place , may be BC to incldue the .h from drybicart
Is this defined elsewhere? We shouldn't define this in multiple places. I think it should be in one header file - maybe balancing chamber controller.
Use the get Acid mix control interval?
Updates the comments for all new variables.
There should be a \b before the type of details (Inputs and Outputs here) for documentation please update for all
Remove blank line
Remove blank line There should be a \b before the type of details (Inputs and Outputs here) for documentation
There should be a \b before the type of details (Inputs and Outputs here) for documentation Inputs: None?
a time out for decaying D66 pressure to 3 PSI after D65 closed
remove space after <= and add space end of function call.
Is this time out after D80 opening or D65 closing state? if it is D65 closed, you need to start the timer after D65 closing.
if we are not checking level, this can be removed and function name also can be updated accordingly.
Is this a TODO? If so, add TODO to the comment and maybe be a little more descriptive of what needs to be done.
can we get 'runPIController' inside deadband control check?
Initialize the 'dryBicartPersistantStartTime' back to zero after timeout done, for the next iterations.
Nope. Since we are here after level is LOW (fill request from balancing chamber control state machine) and not filled yet. So level is still LOW
Can we check payload.index is equal to D6_LEVL (instead of above comparision)?
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.