Zoltan Miskolci

- Matching alarm publish name with variable name

- Updated test config names - Removed invalid publish from gen_dialysate

    • -2
    • +2
    /leahi_dialin/common/test_config_defs.py
    • -4
    • +0
    /leahi_dialin/dd/modules/gen_dialysate.py
When building locally without all the parameters the generated name throws an error during python pip install. So this ensures that if no parameter is added, it uses the default name which is compa...

When building locally without all the parameters the generated name throws an error during python pip install.
So this ensures that if no parameter is added, it uses the default name which is compatible with pip install.
The " -+. " sequence is throwing an error.

removed commented out line

removed commented out line

removed

It's a class containing all the enums of the def file. Yes we need them as many other Framework and huge amount of code are using them.

It's a class containing all the enums of the def file.
Yes we need them as many other Framework and huge amount of code are using them.

Obviously not. You were reminding me about backwards compatibility. This variable and class ensures that even with this major refactor to the enum structure, the current codebase remains working.

Obviously not.
You were reminding me about backwards compatibility. This variable and class ensures that even with this major refactor to the enum structure, the current codebase remains working.

Yes, they are aliases that are used for example in Dialin's temperature sensor module to decide which msg id to use for the override. Updating an alias's value when adding new entries to the enum i...

Yes, they are aliases that are used for example in Dialin's temperature sensor module to decide which msg id to use for the override.
Updating an alias's value when adding new entries to the enum is easier then changing the last or first sensor type across Dialin modules, which could be forgotten or missed.

DialinEnum from_str function using it to match a strung to an enum member. Converting the string to enum for cleaner code in test cases.

DialinEnum from_str function using it to match a strung to an enum member. Converting the string to enum for cleaner code in test cases.

Yes

Yes

Based on the msg id, it should be here

Based on the msg id, it should be here

- fixed valve_state_enums to be updated - Fixed balancing chamber broadcast processing

Please pull the latest staging and update the test configs regarding that. I think the dry bicart test config is already there

Please pull the latest staging and update the test configs regarding that.
I think the dry bicart test config is already there

Merge branch 'staging' into LDT-2787-fp_dd_proxy_commands

    • -109
    • +113
    /leahi_dialin/fp/filtration_purification.py
- added new option to from_str - Fixed dd valve ids - Added a new str to pregen option

fixed renaming

- fixed event, removed TPAR Op mode

removed from string conversion too

Removed

Removed

removed tMODE_TPAR from TDOpModes

Added from_str to td_def enums

    • -242
    • +319
    /leahi_dialin/common/td_defs.py
convert half of td_defs

    • -446
    • +470
    /leahi_dialin/common/td_defs.py
- Updated dd_Defs to use static enums - Switched auto to integers for easier identification during debug - Changed name generation in build script to remove errors from python about name

    • -740
    • +851
    /leahi_dialin/common/dd_defs.py
    • -134
    • +134
    /leahi_dialin/common/fp_defs.py
fp_defs conversion to static enums

    • -484
    • +510
    /leahi_dialin/common/fp_defs.py
added from_str to td_defs

    • -285
    • +410
    /leahi_dialin/common/td_defs.py
from_str added to dd_def enums

    • -377
    • +459
    /leahi_dialin/common/dd_defs.py
Added from_str to fp_def enums

    • -0
    • +69
    /leahi_dialin/common/defs_base.py
    • -213
    • +262
    /leahi_dialin/common/fp_defs.py