Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
I don't need to edit this file. I will check with Dara to exclude this file from code review.

I don't need to edit this file. I will check with Dara to exclude this file from code review.

only maximum 30 float data is allowed. If that is the case then I cannot broadcast the acid or bicarb control signals. need to split it or need two message id.

only maximum 30 float data is allowed. If that is the case then I cannot broadcast the acid or bicarb control signals.
need to split it or need two message id.

Its a problem . Vinay has to decide it

Its a problem . Vinay has to decide it

Algin "=" to be consistent with those below.

Algin "=" to be consistent with those below.

Add blank line after declarations.

Add blank line after declarations.

Add blank line after declarations.

Add blank line after declarations.

Align comment.

Align comment.

Is it time to restore?

Is it time to restore?

Remove extra space after "=".

Remove extra space after "=".

Are we going to restore this or not?

Are we going to restore this or not?

Are we going to restore this or not?

Are we going to restore this or not?

Are we going to restore this or not?

Are we going to restore this or not?

Remove this.

Remove this.

Remove this.

Remove this.

Remove spaces before "=" or align "=".

Remove spaces before "=" or align "=".

Fix comment alignment.

Fix comment alignment.

Why are we commenting this out?

Why are we commenting this out?

Is it time to restore now?

Is it time to restore now?

Can we remove these 2 lines of code?

Can we remove these 2 lines of code?

Remove extra blank lines.

Remove extra blank lines.

This "s" looks out of place.

This "s" looks out of place.

But this is a f/w shared unit (so both TD and DD f/w include this unit in their build) and we may add a case later. Because it's a common unit, we should design it to work for both f/w builds.

But this is a f/w shared unit (so both TD and DD f/w include this unit in their build) and we may add a case later. Because it's a common unit, we should design it to work for both f/w builds.

No, its deleted

No, its deleted

I do not see TD use case in any of the PI controller functions. so do we need TD here ?

I do not see TD use case in any of the PI controller functions. so do we need TD here ?

LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
Why do we have a local sub state variable that we don't do anything with?

Why do we have a local sub state variable that we don't do anything with?

This is to avoid compilation warning

This is to avoid compilation warning

I don't think these coefficient access functions need to be public. Consider making them private.

I don't think these coefficient access functions need to be public. Consider making them private.

These seem more like concentrate doses, not dialysate doses. Consider renaming.

These seem more like concentrate doses, not dialysate doses. Consider renaming.