ConcentratePumps.c

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
LEAHI-DD-FIRMWARE-LDT-2004_Dialysate Composition - DD
Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

  1. … 5 more files in changeset.
LDT-3750 updated the balancing chamber and concentrate pumps

  1. … 1 more file in changeset.
Add TODO comment to restore later.

Add TODO comment to restore later.

LEAHI-DD-FIRMWARE-LDT-3750_DD_Staging_Rebase
LEAHI-DD-FIRMWARE-LDT-3750_DD_Staging_Rebase
updated.

updated.

updated.

updated.

done.

done.

4 bytes reg.

4 bytes reg.

4 byte reg.

4 byte reg.

Remove typecast.

Remove typecast.

LEAHI-DD-FIRMWARE-LDT-3628_Request to change the target rev count for piston pumps from 2 byte to 4 byte register
LEAHI-DD-FIRMWARE-LDT-3628_Request to change the target rev count for piston pumps from 2 byte to 4 byte register
Looked better before - now misspelled.

Looked better before - now misspelled.

0.0F

0.0F

indent looks 1 space off.

indent looks 1 space off.

If we include math, I like to see comment explaining what we're doing with it. Why remove it?

If we include math, I like to see comment explaining what we're doing with it. Why remove it?

Declare at top of scope, assign down here if needed.

Declare at top of scope, assign down here if needed.

If there is only one target temperature, why are we passing in a heater ID to function? This seems weird to me having both heaters share a target.

If there is only one target temperature, why are we passing in a heater ID to function? This seems weird to me having both heaters share a target.

Remove blank line.

Remove blank line.

Which heater does single target temp apply to? Or does same target apply to both heaters?

Which heater does single target temp apply to? Or does same target apply to both heaters?

I don't see ufVolumeperIteration being set in this function - why did you add it to Outputs?

I don't see ufVolumeperIteration being set in this function - why did you add it to Outputs?

Remove blank line.

Remove blank line.

return should be "none"

return should be "none"

Why 2 Inputs?

Why 2 Inputs?

return should be "none"

return should be "none"

return should be "none"

return should be "none"

Why 2 inputs?

Why 2 inputs?

Why 2 Inputs?

Why 2 Inputs?

Do we really need 64 bits for this?

Do we really need 64 bits for this?