Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-7802: Used test support get override value

    • -30
    • +2
    ./Controllers/ConductivitySensors.c
  1. … 3 more files in changeset.
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done.

Done.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done.

Done.

DEN-7802: Addressed code review comments

Done.

Done.

Done.

Done.

Done.

Done.

I would prefer we return a TRUE or FALSE explicitly (like function below).

I would prefer we return a TRUE or FALSE explicitly (like function below).

Removed.

Removed.

DEN-7802: Removed concentrate pump voltage broadcast data

Start comments w/ capital letter.

Start comments w/ capital letter.

I know bicarb pump "on" request above does not immediately turn the pump on, but I still think we should set target speed before we request pump on.

I know bicarb pump "on" request above does not immediately turn the pump on, but I still think we should set target speed before we request pump on.

I know acid pump "on" request above does not immediately turn the pump on, but I still think we should set target speed before we request pump on.

I know acid pump "on" request above does not immediately turn the pump on, but I still think we should set target speed before we request pump on.

Changes "Pumps" to "Pump". Request is for single pump.

Changes "Pumps" to "Pump". Request is for single pump.

So we are not monitoring these, but we are publishing them?

So we are not monitoring these, but we are publishing them?

DEN-7802: Removed concentrate pumps voltage monitor and fixed pressure checks

Merge branch 'staging' into DEN-7802-dg_dev-dialysate-generation

    • -5
    • +128
    ./Controllers/ConcentratePumps.c
    • -5
    • +108
    ./Controllers/ConductivitySensors.c
    • -100
    • +109
    ./Controllers/TemperatureSensors.c
  1. … 7 more files in changeset.
DEN-7802: Removed unused build flag

Removed as it is not needed anymore.

Removed as it is not needed anymore.

Renamed to match new alarm ID.

Renamed to match new alarm ID.

DEN-7802: Removed V2 system define