Lines of code count not available
Which alarm?
bpPublishTimerCtr is the output?
Input is none. Add @Alarms
Should we list the alarm ids? @Alarms?
requestAbortBPReading is the output?
requestPedsBPReading is the output?
requestAdultBPReading is the output?
pendingVitalsRequest is the output?
We should initialize in the init function?
bpMeasurementReady is an output?
nextState is the return value. Should we add requestAdultBPMeasurement, bpMeasurementReady and bpDriverError to outputs?
Where are we updating the bpResults?
We should initialize this in init functions.
please add a comment
pressure is a param?
command is a param?
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.
Updated
Removed
Fixed
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.