BloodFlow.c

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Do we need to send UI a response? Accept/reject?

Do we need to send UI a response? Accept/reject?

Is this a message handler? If so, parameter should be a message and option should be extracted from message payload.

Is this a message handler? If so, parameter should be a message and option should be extracted from message payload.

Does this command cause DD to transition to standby? It looks like the command to give when we transition into post treatment mode to tell DD to transition to post-gen dialysate mode.

Does this command cause DD to transition to standby? It looks like the command to give when we transition into post treatment mode to tell DD to transition to post-gen dialysate mode.

Switch not needed here. Just signal standby mode with the received selection.

Switch not needed here. Just signal standby mode with the received selection.

Should be disinfection selection type, not command type. Also, move this down into if statement scope.

Should be disinfection selection type, not command type.
Also, move this down into if statement scope.

Why are we signaling a command to standby mode? We should signal the selection that we received from UI. Standby mode (later when we get there) will then look at the selection and decide there whic...

Why are we signaling a command to standby mode? We should signal the selection that we received from UI. Standby mode (later when we get there) will then look at the selection and decide there which command to send to DD.

Remove blank line.

Remove blank line.

Won't this happen when we transition to auto-eject state? Why do it here?

Won't this happen when we transition to auto-eject state? Why do it here?

Shouldn't auto-eject state handle this BP homing?

Shouldn't auto-eject state handle this BP homing?

Why are we worried about "handled"? Once confirmed, should we just go to auto eject state?

Why are we worried about "handled"? Once confirmed, should we just go to auto eject state?

Why is this if statement separated from the one above? Why not just do this stuff in that if statement?

Why is this if statement separated from the one above? Why not just do this stuff in that if statement?

There should be function(s) in DDInterface to tell DD what we want (bypass, Qd, etc...).

There should be function(s) in DDInterface to tell DD what we want (bypass, Qd, etc...).

This publish code should be in a publish function and we just call that function here.

This publish code should be in a publish function and we just call that function here.

Add comment to right.

Add comment to right.

Why is this above the definitions banner?

Why is this above the definitions banner?

This signal function looks like a message handler. If so, it should take a message (like handlers below) and get the option from the message payload.

This signal function looks like a message handler. If so, it should take a message (like handlers below) and get the option from the message payload.

Needs comment to right.

Needs comment to right.

Is TD going to request active cooling or is it going to happen automatically after heat or heat+citric disinfection completes? You have an enum of heat disinfection commands, so why do we need sepa...

Is TD going to request active cooling or is it going to happen automatically after heat or heat+citric disinfection completes?
You have an enum of heat disinfection commands, so why do we need separate messages for each disinfection? Should just be 1 message to start a disinfection with a command enum as its payload.

fix the id back to 255 in the struct

fix the id back to 255 in the struct

Not needed? Add another BOOL for disinfection option received.

Not needed? Add another BOOL for disinfection option received.

Initialize this.

Initialize this.

Remove TODO

Remove TODO

Remove disable of ejector motor.

Remove disable of ejector motor.

Remove RotaryValve.h. It's obsolete and we should be using higher level controller Valves.h anyway.

Remove RotaryValve.h. It's obsolete and we should be using higher level controller Valves.h anyway.

Just move msg handlers from Messaging to here so you don't need a signal.

Just move msg handlers from Messaging to here so you don't need a signal.

Move to .c file to make private.

Move to .c file to make private.

Not needed.

Not needed.

Missing heat citric

Missing heat citric

Put back to reserved

Put back to reserved