Comment Search

Put blank line back.

Add a TODO comment to adjust this when new syringe pump assembly is ready.

Fix indent for break. Add comment before break "// unhandled position - do nothing.

Are we done testing? Can we remote test variables and code?

When we send any message to UI, usually add the message id in the doxygen header like this.

• @details \b Message \b Sent: MSG_ID_TD_RESP_TREATMENT_PARAMS_TO_VALIDATE

Please refer to the function sendTreatmentParamsResponse()

Revert the staging test script and using the different branch

Add @message MSG_ID_TD_RECIRCULATE_RESPONSE

I thought Eliza said they wanted to do blood sitting timer even if blood was rinsed back (still a little blood in line).

Fixed

Inputs : fpgaSensorReadings
Outputs: none

Add cumulativeBloodPrimeVolume_mL to ouputs.

Optional: While you are updating, can you please also add bloodPrimeRampFlowRate_mL_min and bloodPrimeState to the inputs?

Space before and after paranthesis.

remove from prime // priming - TODO check if needed. may not need

You need to initialize the valve states and timer etc., as similar to when you jump from idle state to fill water state. Fill water state just checks the pressure check condition.

Remove extra blank line.

Add comment.

Use if else or ternary operator to set BOOL to TRUE or FALSE explicitly.

Removed

fixed thanks

fixed thanks

Do we need to transition to next state DD_PREGEN_DRY_BICART_INITIAL_FILL here?

Does default mean invalid state (s/w fault) or does it mean it's a state that we don't need to do anything for? If latter, add a comment here saying that.

fixed thanks

Or User cancels drain request: To cancel the ongoing drain process. Could you please add the condition as well?

fixed thanks