firmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
LDT-3103 updated fill timeout alarm logic and handling

    • -12
    • +3
    ./App/Controllers/BalancingChamber.c
LDT-3103 updated fill timeout alarm logic and handling

    • -12
    • +3
    ./App/Controllers/BalancingChamber.c
LDT-1195: Updated function comments and requestBalChamberSwitching

    • -4
    • +12
    ./App/Controllers/BalancingChamber.c
Remove extra blank line.

Remove extra blank line.

Should we create a structure for these moving average filters and make an array of them based on enum of temp sensors? It looks kind of messy having separate handling for each sensor - can we conso...

Should we create a structure for these moving average filters and make an array of them based on enum of temp sensors?
It looks kind of messy having separate handling for each sensor - can we consolidate some of this code?

Should we maybe just have a constant array of sample counts based on enum of temp sensors?

Should we maybe just have a constant array of sample counts based on enum of temp sensors?

Should min and max be properties of the controller instead of global (assumes all controllers will want same min/max)? Controller properties already include a min and max field - can we use those?

Should min and max be properties of the controller instead of global (assumes all controllers will want same min/max)?
Controller properties already include a min and max field - can we use those?

Remove blank line.

Remove blank line.

changed

changed

removed from everywhere

removed from everywhere

removed

removed

LDY-3103 addressed code review comments

    • -20
    • +1
    ./App/Controllers/BalancingChamber.c
LDY-3103 addressed code review comments

    • -20
    • +1
    ./App/Controllers/BalancingChamber.c
Bamboo Commit: Updated DDCommon.h with build versions from Bamboo.

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

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

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

Result is a BOOL variable, use FALSE.

Result is a BOOL variable, use FALSE.

updated.

updated.

done.

done.

done

done

changed accordingly.

changed accordingly.

LDT-3103 addressed code review comments

    • -8
    • +21
    ./App/Controllers/BalancingChamber.c
LDT-3103 addressed code review comments

    • -6
    • +19
    ./App/Controllers/BalancingChamber.c
Fix getFreshDialPumpInitialRpm function name and brief inputs comment

LDT-3750 cherry-picked ca27c61

I think the 2 initial rpm constants are inputs. The test config we would normally not include.

I think the 2 initial rpm constants are inputs. The test config we would normally not include.

I think the word "Initial" should be in the function name (i.e. getFreshDialPumpInitialRpm).

I think the word "Initial" should be in the function name (i.e. getFreshDialPumpInitialRpm).