Sean Nash

No need to initialize these to zero.

No need to initialize these to zero.

Remove extra blank line.

Remove extra blank line.

Remove extra blank line.

Remove extra blank line.

Remove blank line.

Remove blank line.

Should we add _TARGET to these names for clarity?

Should we add _TARGET to these names for clarity?

Is the count really a float?

Is the count really a float?

Do we need an override for the RPM feedback?

Do we need an override for the RPM feedback?

This is a maximum per treatment?

This is a maximum per treatment?

I'm not seeing fixes in this file.

I'm not seeing fixes in this file.

Remove extra blank line.

Remove extra blank line.

Is this temporary? If so, put a TODO to remove later.

Is this temporary? If so, put a TODO to remove later.

Remove extra spaces between { and msg name.

Remove extra spaces between { and msg name.

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC e...

Is this always needed or is it only needed in one or more specific modes (i.e. gen dialysate mode)? If only needed in specific modes, I think this should be called from those modes (similar to BC exec).

Do these belong in the spent chamber fill group?

Do these belong in the spent chamber fill group?

Add space before ).

Add space before ).

Add space between if and (.

Add space between if and (.

Be consistent on whether you have a blank line between break and next case.

Be consistent on whether you have a blank line between break and next case.

Keep blank line here.

Keep blank line here.

What are these coefficients for? At least describe in comments, if not the names.

What are these coefficients for? At least describe in comments, if not the names.

Remove extra blank line.

Remove extra blank line.