Is this the right time to zero this volume?

update total volume too?

maybe use target rate for both.

Use #define for array size.

Make the 6 a #define

only 1 return statement

remove comma

return FALSE on else

signal functions should return BOOL.

Move up with rest

Move this function to your service.

Add HDF state too

Not needed.

Should always exec your bolus sm.

Keep 2 blank lines before banner

Add fluid bolus state

Add fluid bolus state

FLUID_BOLUS_CMD_EVENT

slaveAddr should be an enum now and so just "device".

Add a banner before Dialin test support functions (see other units for example).

Initialize .override to OVERRIDE_RESET

Why is this commented out? Do we not need to initialize it?

Rename similar to ADDR definitions below. these are not commands.

Combine charger and pack data into one payload with one broadcast message.

Move these 3 override messages (should end in OVERRIDE_REQUEST) to TD Dialin section below and assign an ID in the 0x8... range.

I think we only need one data message for both charger and battery pack data.

Remove.

uncomment.

Why was this function header changed? I think it was correct before.

If these messages are for change to UF volume during treatment, the message ID should have the word TREATMENT in it to distinguish from the original UF volume message that comes in pre-treatment.