Comment Search

add changing the the opmode first too.. td.td_operation_mode( TDOpModes.MODE_TREA.value, 0 )

Replace current with torque.

I don't think these min/max (for control output in output units i.e. PWM) are appropriate for limiting maximum error sum step size (error is in feedback/target units i.e. RPM).

I don't think we have motor current. We're getting torque instead. Just replace current with torque.

Can we delete this commented out line?

I think this validate function needs to know the requested mode as well - should have 2 params.

Aligned.

Please align all as following:

        { "arterial_pressure"    : 100,  "arterial_limit_low" : -300, "arterial_limit_high" : 120, 
          "venous_pressure"      : 140,  "venous_limit_low"   : -100, "venous_limit_high"   : 250,
          "pressure_limit_state" : 2,    "tmp_pressure"       : 250 , "tmp_limit_low"       : -400 , "tmp_limit_high" :  200 },

dialyste_tmp_title_Text

If this is specific to only your test case then move this in the test case

Match properties from Denali where you can. The priorities should both be HIGH. The rank for the warning should be a higher number. The alarm should be TRUE for no resume and no rinseback. So, I know these properties aren't correct.
One thing I don't think we should copy from Denali though - I think the warning should not be TRUE for the STOP property so that we stay in blood prime state.

This message ID (0xF0) is no longer valid? and remove empty spaces.

Assign bloodPrimeStartMS here to mark start of pause.

Add '#' to start of line?

Align comment.

remove all the waitForGUI

BloodFlow.c should have a public get function for torque like other overrides.

I think we should replace motor current with torque instead of adding torque as new field.

Update the comments = Valve 7, D26.

Changed and left them as static, as we don't need these outside this unit

Can we delete this commented out line?

if ( TRUE == result )

yeah its obsolete. This is an old branch that recently got synced with latest staging. It looks like the CR doesn't recognize the correct changes from staging. There should only be 1 file change ( conductivity sensors ).

done.

4 byte reg.