Lines of code count not available
please add message id.
We should set all TD actuators here, not just blood pump.
fpgaActuatorSetPoints is an output. Input is none.
I think we should accept this as long as we're in pre-tx: install state. User might confirm before auto-load finishes and that's ok.
Change "SetupConnection" to "SetupTubingSetConnection"
Recommend changing "SETUP_CONNECTION" to "SETUP_TUBING_SET_CONNECTIONS" in these 2 msg ID names.
Include "BPModule.h".
BPModule is just one of the controllers - move this up into the controllers section.
We should not comment this out.
It's still here. Do we need to initialize it here too or is it sufficient to initialize once from main at startup?
We are already executing this from TaskGeneral.c. Remove from here.
Monitor is mis-spelled. Should be Control anyway. Move prior to execAlarmMgmt() above.
I think all flags are outputs (not just adult).
These look like outputs to me (not inputs).
Added
Bubble status, Low and high level level sensors.
Input : Blood pump door status?
To be restored?
Are these being initialized in the init function?
Need to resolve the merge issues?
remove this
remove this case
align the comment part
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.