leahi-common

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-5472: Added Mesg Id to trigger Barcode Request through dialin

Yes,

Yes,

In pre-treatment

In pre-treatment

Fixed

Fixed

Yes, fixed

Yes, fixed

Add a command to turn air pump off here in this state (since we aren't doing that in transition anymore to prevent large pressure drop).

Add a command to turn air pump off here in this state (since we aren't doing that in transition anymore to prevent large pressure drop).

Don't turn off air pump here - looks like we lose too much air pressure when we do that. Move this command to the stabilize state handler (I'll add a comment there).

Don't turn off air pump here - looks like we lose too much air pressure when we do that. Move this command to the stabilize state handler (I'll add a comment there).

You removed it here, but where did it go? I expected to see it in next case below.

You removed it here, but where did it go? I expected to see it in next case below.

LDT-5613: Changes in Pressure self test

Align "="

Align "="

Add comment to right.

Add comment to right.

Better to say != TRUE here.

Better to say != TRUE here.

Add TODO comment as to whether we should trigger an alarm/fault here.

Add TODO comment as to whether we should trigger an alarm/fault here.

Add blank line after declaration. Fix indent.

Add blank line after declaration.
Fix indent.

We can un-comment syringe pump stop now.

We can un-comment syringe pump stop now.

I think this function should be private (called by public message handler).

I think this function should be private (called by public message handler).

Should this be static (private)?

Should this be static (private)?

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.

Added alarm for disinfect in progress

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.

LDT-3992: Merge latest staging & reolve merge conflicts

merge conflicts resolved

LDT-4024 added msg id's for broadcast

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.