leahi-fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-3103 addressed code review comments and changes

I don't see a TODO. If we intend to uncomment this line out later, we should add a TODO comment to the right as a reminder. If we do not intend to uncomment this later, we should just delete it.

I don't see a TODO. If we intend to uncomment this line out later, we should add a TODO comment to the right as a reminder. If we do not intend to uncomment this later, we should just delete it.

Fix alignment of "=".

Fix alignment of "=".

Add a param line in this header.

Add a param line in this header.

Can we delete this commented out line?

Can we delete this commented out line?

Add a params line in function header.

Add a params line in function header.

Why did we change this spelling? Now it's wrong.

Why did we change this spelling? Now it's wrong.

Can we delete this commented out line?

Can we delete this commented out line?

Keep this blank line.

Keep this blank line.

Should be before ModeStandby.

Should be before ModeStandby.

I see we are doing this same logic in multiple places. Can we just have a function that returns the minimum speed for the given dialysate pump?

I see we are doing this same logic in multiple places. Can we just have a function that returns the minimum speed for the given dialysate pump?

Remove blank line.

Remove blank line.

Remove blank line.

Remove blank line.

Should assign a local variable to this equation and then return the variable. Can this equation result in a negative number? If so, we should handle that first before converting to unsigned int. Al...

Should assign a local variable to this equation and then return the variable.
Can this equation result in a negative number? If so, we should handle that first before converting to unsigned int.
Also, do we want to do any rounding when converting float to integer?

Remove extra blank line.

Remove extra blank line.

The TODO has already been mentioned in staging. They have it commented out for strike and hold behavior.

The TODO has already been mentioned in staging. They have it commented out for strike and hold behavior.

Add TEST_CONFIG_TESTING_WITHOUT_TUBING_SET test configuration enum.

LEAHI-TD-FIRMWARE-LDT-3600_Request for TD Test Config for Testing without Tubing Set on TD
LEAHI-TD-FIRMWARE-LDT-3600_Request for TD Test Config for Testing without Tubing Set on TD
Merge branch 'develop' of bitbucket.org:diality-cloud/leahi-fwcommon into develop

Merge branch 'LDT-3103-implement-closed-loop-control-f' into develop

LDT-3103 merged latest staging

Merge branch 'staging' into LDT-2004-v2-conductivity-sensors

Merged staging

LDT-3505: Added Test Config for Syringe Pump

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

LDT-3605

LDT-3605 updated the test support module