DryBiCart.c

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
All these are more of debug related overrides (acid and bicarb P & I gain). you can keep them in one function, when the implementation is complete, you may remove these. The init function is too bi...

All these are more of debug related overrides (acid and bicarb P & I gain). you can keep them in one function, when the implementation is complete, you may remove these. The init function is too big, group them and split in the functions and call here.

Some of these parameters can be changed over td proxy commands, need not to implement additional overrides.

Some of these parameters can be changed over td proxy commands, need not to implement additional overrides.

I guess all these MACRO implementations should move to function/s to compute the final acid and bicarb volume, that would help to calculate the volumes when Na and Bicarb params changed during trea...

I guess all these MACRO implementations should move to function/s to compute the final acid and bicarb volume, that would help to calculate the volumes when Na and Bicarb params changed during treatment.

looks Bicarb and sodium settings interchanged? There are multiple macros on the same values. remove the duplicates.

looks Bicarb and sodium settings interchanged? There are multiple macros on the same values. remove the duplicates.

33.0?

33.0?

Add comment for each macro.

Add comment for each macro.

For all floats add F at the end, across the file.

For all floats add F at the end, across the file.

Align the comment

Align the comment

There is a #define for BAL_CHAMBER_FILL_VOLUME_ML in BalancingChamber.h. Why can't we just use that instead of redefining it here?

There is a #define for BAL_CHAMBER_FILL_VOLUME_ML in BalancingChamber.h. Why can't we just use that instead of redefining it here?

This dead band control ? here is the ideal place , may be BC to incldue the .h from drybicart

This dead band control ? here is the ideal place , may be BC to incldue the .h from drybicart

Is this defined elsewhere? We shouldn't define this in multiple places. I think it should be in one header file - maybe balancing chamber controller.

Is this defined elsewhere? We shouldn't define this in multiple places. I think it should be in one header file - maybe balancing chamber controller.

LEAHI-DD-FIRMWARE-LDT-4246_FW Support for UI Demo
LEAHI-DD-FIRMWARE-LDT-4246_FW Support for UI Demo
Use the get Acid mix control interval?

Use the get Acid mix control interval?

Updates the comments for all new variables.

Updates the comments for all new variables.

There should be a \b before the type of details (Inputs and Outputs here) for documentation please update for all

There should be a \b before the type of details (Inputs and Outputs here) for documentation please update for all

Remove blank line

Remove blank line

Remove blank line There should be a \b before the type of details (Inputs and Outputs here) for documentation

Remove blank line
There should be a \b before the type of details (Inputs and Outputs here) for documentation

There should be a \b before the type of details (Inputs and Outputs here) for documentation Inputs: None?

There should be a \b before the type of details (Inputs and Outputs here) for documentation
Inputs: None?

LEAHI-DD-FIRMWARE-LDT-4183_Dialysate Composition (Dry Bicarbonate) - FW Implementation - 2/1:
LEAHI-DD-FIRMWARE-LDT-4183_Dialysate Composition (Dry Bicarbonate) - FW Implementation - 2/1:
LEAHI-DD-FIRMWARE-LDT-3978_Switch Chamber F conductive level sensors to broadcast 1 as sensing liquid
LEAHI-DD-FIRMWARE-LDT-3978_Switch Chamber F conductive level sensors to broadcast 1 as sensing liquid
LDT-3978-midifying drybicart supply upper threshold pressure

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

  1. … 12 more files in changeset.