hdfirmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-9906: Added test code to calculate and report measured UF rate based on change in UF volume.

    • -0
    • +14
    /firmware/App/Controllers/DialOutFlow.c
DEN-9906: Removed obsolete blood flow code.

DEN-9906 Fixed merge conflict

    • -124
    • +1
    /firmware/App/Controllers/DGInterface.c
    • -42
    • +1
    /firmware/App/Controllers/DGInterface.h
    • -21
    • +7
    /firmware/App/Services/SystemComm.c
DEN-9906 Implement HD dialysate flow sensor

    • -0
    • +32
    /firmware/App/Controllers/DGInterface.c
    • -2
    • +10
    /firmware/App/Controllers/DGInterface.h
DEN-9906: Added overrides for blood pump rotor count and arterial pressure offset. Removed unused handling of DG data and related functions/variables. Fixed DEN-10973.

    • -335
    • +31
    /firmware/App/Controllers/BloodFlow.c
    • -123
    • +0
    /firmware/App/Controllers/DGInterface.c
    • -41
    • +0
    /firmware/App/Controllers/DGInterface.h
    • -3
    • +49
    /firmware/App/Controllers/PresOccl.c
    • -20
    • +6
    /firmware/App/Services/SystemComm.c
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.

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.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

RESOLVED IN CODE WALKTHROUGH.

DEN-9906: Addressed code review comments.

    • -0
    • +16
    /firmware/App/Controllers/Switches.c
    • -1
    • +16
    /firmware/App/Controllers/Switches.h
    • -1
    • +16
    /firmware/App/Controllers/Temperatures.c
    • -0
    • +16
    /firmware/App/Controllers/Temperatures.h
Fixed.

Fixed.

Fixed.

Fixed.

If using a "==" comparison operator, I like to put the literal first so that the compiler will error if I accidentally use "=" instead. Otherwise, I like to put the literal last because it reads be...

If using a "==" comparison operator, I like to put the literal first so that the compiler will error if I accidentally use "=" instead.
Otherwise, I like to put the literal last because it reads better.

I was thinking 150 was perhaps too fast for UI. But you're right - that only seems to be true at startup. I'll put back to 150.

I was thinking 150 was perhaps too fast for UI. But you're right - that only seems to be true at startup. I'll put back to 150.

This toggle macro is designed to be generic - so does not assume you are toggling between 0 and 1. Could be toggling between ON and OFF or OPEN and CLOSED, etc... So you have to give the macro the ...

This toggle macro is designed to be generic - so does not assume you are toggling between 0 and 1. Could be toggling between ON and OFF or OPEN and CLOSED, etc... So you have to give the macro the two values that you are toggling between.

FALSE is simply zero and TRUE is one. In case a boolean is somehow corrupted to something other than 1 or 0, I like to err on the side of safety. So the way I have it now, if isBloodPumpRunning() g...

FALSE is simply zero and TRUE is one. In case a boolean is somehow corrupted to something other than 1 or 0, I like to err on the side of safety.
So the way I have it now, if isBloodPumpRunning() gets corrupted to > 1, it will consider that the pump is not running which is the safer thing to do.

currentSubMode is already a U32 - no cast needed.

currentSubMode is already a U32 - no cast needed.