rejReason = INVALID REQUEST FORMAT

Missing comment.

update comment. next one as well.

blank line after the banner and maintain alphabetical order of include files.

I think these remaining 4 else ifs need to have another && condition that your service is in idle state so that we don't change back into one of these 4 states while a bolus is in progress.

Can we restore the real code here yet? Ask Dara.

maybe use target rate for both.

Use #define for array size.

Make the 6 a #define

only 1 return statement

remove comma

signal functions should return BOOL.

Move up with rest

Move this function to your service.

Not needed.

Should always exec your bolus sm.

Keep 2 blank lines before banner

Add fluid bolus state

Add fluid bolus state

We should just have 1 of these. Let's have TD always working in mL and mL/min, and we'll just convert before sending to UI or after receiving from UI.

we don't need pragma alignment on enums, could you exclude it ?

Updated to ml/min, except for broadcast in treatment mode

What TD does with UF rate is mostly to tell DD what rate we want. DD wants mL/min. TD can keep L/hr and convert to mL/min every time it needs to send DD a message or it can keep mL/min and convert to L/hr every time it needs to send UI a message. Either way works for me.

Changed to U32