MsgDefs.h

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
BiCarb related state machines will be called in pre-gen, gen dialysate and post-gen modes. currently its kept here, so any modes can set a flag to initiate the required operations.

BiCarb related state machines will be called in pre-gen, gen dialysate and post-gen modes. currently its kept here, so any modes can set a flag to initiate the required operations.

Done.

Done.

Done.

Done.

Updated.

Updated.

Done.

Done.

Done.

Done.

Done.

Done.

We will move towards 200 RPM being the minimum speed with updated Maxon controller. So, there is no need to maintain two different minimum RPM going forward.

We will move towards 200 RPM being the minimum speed with updated Maxon controller. So, there is no need to maintain two different minimum RPM going forward.

Done.

Done.

Done.

Done.

We tend to initialize the local variables as a standard practice.

We tend to initialize the local variables as a standard practice.

These don't need to be initialized to zero.

These don't need to be initialized to zero.

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done

Done.

Done.

No. It is permanent, I would like to know the value for the homing.

No. It is permanent, I would like to know the value for the homing.

The else in this trinary looks wrong - doesn't round up to multiple of 8.

The else in this trinary looks wrong - doesn't round up to multiple of 8.

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

This if should have an else where we zero .valveOpsStartTime (in case it briefly got set but then failed this if before timeout).

Declarations should be at top of scope.

Declarations should be at top of scope.

Is this temporary?

Is this temporary?

Done

Done

This function has an output.

This function has an output.

Remove extra blank line.

Remove extra blank line.

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

Not sure how Doxygen will handle this comment between the /// and the const. I think we should remove it.

Done

Done