Comment Search

With the updated FPGA logic for continuous flow rate, doesn't need this variable anymore.

Done.

Done.

Update brief and param to say that this function is only operating on the given pump. This and all functions with pump ID as param.

Default for spacing is 0, so this can be removed.

removed. thanks!

fixed thanks!

added. thanks!

fixed. thanks!

it should be okay if we call when were not paused.. The idea is that if it was called the FP would ignore it since it was already in flush.

Changed the function to return a boolean.

fixed. thanks!

removed

If this property is not being used, then can it be eliminated for now?

Remove blank line.

Do we have an enum for this message defined?

I don't see Dialin override functions for these. Are you leaving these for Michael?

Yes, I will have controller/monitor call these in their init functions which will be called here. But until I have those higher level units, I don't want things to go uninitialized for now.

Not many submodes in pre-tx that don't have substates - maybe cartridge install. Should call this when entering a submode with no substates.

RESOLVED

Done.

Use \\\<.

Vectorcast won't be able to reach inside this if statement. It's dead code for now. Maybe better to just comment it out.

We are (I think now) allowing door open in first 2 states (UF and invert dialyzer), so should not be getting door open alarm when UI is prompting user to invert dialyzer. When we get to 3rd state (where we start requiring door to be closed), we are looking for user to connect patient and confirm. UI should be following along with our states and showing appropriate screens.
RESOLVED in CODE WALKTHROUGH.

Fixed. Thanks!

This looks wrong.