dd-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Ok to reject for now (because we don't want TD to go to disinfection state while DD doesn't). Add TODO to handle other disinfection options when they are implemented.

Ok to reject for now (because we don't want TD to go to disinfection state while DD doesn't). Add TODO to handle other disinfection options when they are implemented.

Add a blank line between banner and comment.

Add a blank line between banner and comment.

gen to post gen and post gen to standby are simple commands unlike the pre-gen and gen dialysate requests which have a bunch of parameters. I think we should keep the simple transition requests in ...

gen to post gen and post gen to standby are simple commands unlike the pre-gen and gen dialysate requests which have a bunch of parameters. I think we should keep the simple transition requests in this enum with the disinfection requests and eventually any other simple command requests.

DD only needs a command to start a specific disinfection. This can be achieved with the one generic command message here.

DD only needs a command to start a specific disinfection. This can be achieved with the one generic command message here.

I was thinking we would have a generic command message with a single enum for the command ID. This one message could handle many different commands including post get to standby command.

I was thinking we would have a generic command message with a single enum for the command ID. This one message could handle many different commands including post get to standby command.

added

added

Add blank line after declarations.

Add blank line after declarations.

LEAHI-DD-FIRMWARE-LDT-5155_(DD) D79 rinse pump does not run when commanded (staging 94+)
LEAHI-DD-FIRMWARE-LDT-5155_(DD) D79 rinse pump does not run when commanded (staging 94+)
Please add another sub condition for now that sub mode is in DD_POST_GEN_DIALYSATE_STATE_COMPLETE. Make sure you transition substate from Post gen start state to complete state for now to wait for ...

Please add another sub condition for now that sub mode is in DD_POST_GEN_DIALYSATE_STATE_COMPLETE. Make sure you transition substate from Post gen start state to complete state for now to wait for this request.

for now, keep this only related for disinfection cmds. for Post gen and standby. Follow the similiar messaging like pre-gen and gen dialysate requests and the reply back. ( but one message/handler )

for now, keep this only related for disinfection cmds.

for Post gen and standby. Follow the similiar messaging like pre-gen and gen dialysate requests and the reply back. ( but one message/handler )

are we rejecting this request for Aug Month demo, I guess, you still need to accept these requests and set the corresponding pending DD disinfect request flags.

are we rejecting this request for Aug Month demo, I guess, you still need to accept these requests and set the corresponding pending DD disinfect request flags.

Do we have a MSG_ID for the selection of disinfection? If not, lets rename and repurpose this one.

Do we have a MSG_ID for the selection of disinfection? If not, lets rename and repurpose this one.

MSG_ID_TD_DD_POSTGEN_STANDBY_REQUEST: to perform post gen and standby mode request transition.

MSG_ID_TD_DD_POSTGEN_STANDBY_REQUEST: to perform post gen and standby mode request transition.

LEAHI-DD-FIRMWARE-LDT-5470_Post Treatment - FW Implementation - 4/3:
LEAHI-DD-FIRMWARE-LDT-5470_Post Treatment - FW Implementation - 4/3:
Add TODO to restore later. Why did this need to be commented out?

Add TODO to restore later. Why did this need to be commented out?

Remove blank line.

Remove blank line.

Restore before merging.

Restore before merging.

flip the logic such that featured is the IF and not the ELSE to avoid having a blank branch statement

flip the logic such that featured is the IF and not the ELSE to avoid having a blank branch statement

Can we add current DD recirculation Dwell Time, WCID recirculation Dwell time variables as well and get the required dialin/automated system testing changes?

Can we add current DD recirculation Dwell Time, WCID recirculation Dwell time variables as well and get the required dialin/automated system testing changes?

Add D92 pump On at 50ml/min.

Add D92 pump On at 50ml/min.

Remove D95 from closed state and add D88 to closed state.

Remove D95 from closed state and add D88 to closed state.

D35, D40, D34,D91,D95 : Open M12, P39 : Closed ( keep a TODO comment to remove them later)

D35, D40, D34,D91,D95 : Open
M12, P39 : Closed ( keep a TODO comment to remove them later)

close D80, D64, D14?

close D80, D64, D14?

D80 : open, D65: Close?

D80 : open, D65: Close?