dd-firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'LDT-3103-implement-closed-loop-control-f' into develop

LDT-3103 fixed lower deadband issues with d48 speed

if ( TRUE == result )

if ( TRUE == result )

Since both states require no special handling, remove breaks and group with default - all modes.

Since both states require no special handling, remove breaks and group with default - all modes.

Remove comment.

Remove comment.

Add comment that all of these states require no special handling.

Add comment that all of these states require no special handling.

I think these functions (validate and signal) should be "test" functions and moved down to Dialin test functions section below - for all modes.

I think these functions (validate and signal) should be "test" functions and moved down to Dialin test functions section below - for all modes.

Add comment that no action is required for these modes - defaulting to invalid request.

Add comment that no action is required for these modes - defaulting to invalid request.

I think this switch should be on requested mode - we haven't changed modes yet.

I think this switch should be on requested mode - we haven't changed modes yet.

We have 1 param (and probably should have 2 params - requested mode and submode).

We have 1 param (and probably should have 2 params - requested mode and submode).

currentMode is an input.

currentMode is an input.

I think this validate function needs to know the requested mode as well - should have 2 params.

I think this validate function needs to know the requested mode as well - should have 2 params.

Are we not supporting sub mode requests for these modes?

Are we not supporting sub mode requests for these modes?

if ( TRUE == testChangeSubMode )

if ( TRUE == testChangeSubMode )

I think these 2 functions are in support of Dialin only, so should be prefixed with "test" as well.

I think these 2 functions are in support of Dialin only, so should be prefixed with "test" as well.

I believe we will want BC control during ISO UF state.

I believe we will want BC control during ISO UF state.

Remove blank line and comment saying we're not doing anything (because now we're doing something).

Remove blank line and comment saying we're not doing anything (because now we're doing something).

isBalChamberSwitchingOffRequested is an output.

isBalChamberSwitchingOffRequested is an output.

Should we set the other switch to FALSE, just in case?

Should we set the other switch to FALSE, just in case?

Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

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

    • -4
    • +78
    /firmware/App/Modes/ModeGenDialysate.c
Merge branch 'develop' of bitbucket.org:diality-cloud/dd-firmware into develop

LDT-3103 merged latest staging

Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

Merge branch 'LDT-1195-dd-is-balancing-chamber-switching' into develop

    • -8
    • +21
    /firmware/App/Modes/ModeGenDialysate.c
    • -11
    • +11
    /firmware/App/Modes/ModeStandby.c
Merge branch 'staging' into LDT-2004-v2-conductivity-sensors

    • -107
    • +51
    /firmware/App/Drivers/ConductivityTeensy.c
LDT-1195: Code cleanup, deleted unused variable pendingBalanceChamberSwOnlyRequest

    • -1
    • +0
    /firmware/App/Modes/ModeGenDialysate.c
LDT-1195: Added a variable to keep track of balancing chamber switching activity.

    • -2
    • +2
    /firmware/App/Modes/ModeGenDialysate.c
    • -0
    • +1
    /firmware/App/Modes/ModePreGenDialysate.c
LDT-3103 fixes for new diener 1000 pump