firmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Bamboo Commit: Updated DGCommon.h with build versions from Bamboo

Merged DENBUG-318

    • -8
    • +24
    ./App/Controllers/ConductivitySensors.c
    • -11
    • +35
    ./App/Services/SystemCommMessages.c
DENBUG-318 added a new event for min inlet water conductivity alarm from HD institutional record

    • -0
    • +1
    ./App/Controllers/ConductivitySensors.c
DENBUG-318 updated the code to receive all the DG institutional record items. Added code to use the min inlet water conductivity alarm from the institutional record

    • -8
    • +23
    ./App/Controllers/ConductivitySensors.c
    • -0
    • +1
    ./App/Controllers/ConductivitySensors.h
    • -11
    • +12
    ./App/Services/SystemCommMessages.c
DENBUG-319 added code to receive a command from dialing to enable the mode fill chemicals test

    • -0
    • +23
    ./App/Services/SystemCommMessages.c
Done

Done

It is an input - don't remove flag there. It is not an output - so remove it there.

It is an input - don't remove flag there. It is not an output - so remove it there.

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

Merge branch 'DENBUG-318-rejection-ratio-setting-feature-implementation-in-manufacturing-configuration-for-treatment-staging' into develop

    • -1
    • +13
    ./App/Controllers/ConductivitySensors.c
DENBUG-322 added code to set CPi and CPo to 20 uS/cm if the values are less than 20 uS/cm

    • -1
    • +13
    ./App/Controllers/ConductivitySensors.c
DENBUG-319 added code to skip the acid and bicarb check in mode fill if it is not the first fill

Done

Done

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

DENBUG-318 changed the location of requesting RO rejection ratio upon starting DG

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

DENBUG-320 addressed a code review comment

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DENBUG-320 brought back the start fill command after removing alarms 281, 282

I don't see the flag being set in this function.

I don't see the flag being set in this function.

Why are we keeping these in the broadcast?

Why are we keeping these in the broadcast?

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

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

Resolved merge conflicts from DENBUG-318

    • -64
    • +47
    ./App/Services/SystemCommMessages.c
DENBUG-318 added the word max to the RO rejection ratio

DENBUG-320 added the word max to the RO rejection ratio

Done

Done

Done

Done

DENBUG-320 removed the code that checked for the acid and bicarb low volume and triggered alarms

    • -64
    • +0
    ./App/Services/SystemCommMessages.c
Should you do something with the received max ratio? It's local, so it's just being dropped.

Should you do something with the received max ratio? It's local, so it's just being dropped.

max ratio

max ratio