Comment Search

Add a comment here to wait for BC switching to go Idle state and transition to spent chamber fill state. Also, add a space after (). Apply the same comment in the next Dialysate delivery state as well.

Don't need this yet.

Can we rename this IsDryBicartChamberFillWindowOpen? ( removing BalChamber)

do you need to add global patientDisconnectConfirmRequest before this line?

update comments

Add blank line after declarations.

fix icon to show no whitespace

group slots at the bottom

Change to start in treatment only

I think most of what's happening in this state should be done in a function in RotaryValve.c that we call from here (since it's doing very low level things that the driver should be doing).

updated and used as suggested.

Add blank line before return.

Do we need to add another else statement and address an alarm if wrong mixId is passed? or Keep only else part instead of else if ( ACID_MIX_ID == mixID) check?? this change may be needed across the file.

updated

Again, ejector should have been retracted way back in pre-treatment mode. We shouldn't need to do it again.

Added

Not necessary if you give retract command on way out of prior state.

These are already in TxParams I believe.

ScrollBarVH

I think ( setTxDur - preStop ) should be changed to ( preStop ).
Also, I think preStop should be renamed to hepDurMin.

Library includes should come first and they should be in <>. Also add comment listing math functions you're using in this unit.

I guess we should have some minimum speed as well as like max speed. Please work with Abhijit on worst case min speed.

remove all commented code.

add offRequestAwaitingUserConfirmation and offRequestPendingTimer to outputs

Vendor - TEL - Arpita Srivastava
please add a space before and after in sizeof(U32) in [sizeof(U32)]