Lines of code count not available
Move to driver.
Recommend command be a U08 here. Driver should have the enum and pass the enum to this function. FPGA should just take a byte.
Move publish call from exec to the check state handle function.
I don't think we need this.
Don't need a flag here. Need a flag in driver and also an isBPCuffConnected() function in driver that we can call from this controller.
Remove.
Just use requestAdultBPReading.
Remove since we're only publishing when we have new measurements.
Done
Use the NEARLY_ZERO constant (instead 0f 0.01F) like the other one.
Add a doxygen comment above this array.
Add a doxygen comment above this array (similar to structures).
Add a doxygen comment above structure for the whole structure (see structure above for example).
Add a doxygen comment for this definition.
This file was updated automatically using HALCoGen.
Missing other test functions in the substitution pump (publish, target speed.)?
update original author name and date.
update all test functions header and descriptions, Input, outputs and param sections.
Are we missing the publish function for the substitution pump?
are we checking any range here? if not, remove the above comment.
Substitution pump. replace across the file.
Remove blank line
Ramp to target speed state is missing?
Can we change the MACRO name to 64 step, if we are using 64 micro steps for HDF?
Do we need to add the D92 pump speed read registers and handle appropriate monitor functions in Substitution Pump controller file?
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.