hdfirmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-7347: Updated temperature sensor publish data structure

    • -17
    • +24
    /firmware/App/Controllers/DGInterface.h
DEN-7347: Addressed code review

    • -3
    • +3
    /firmware/App/Controllers/SyringePump.c
RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

DEN-7347: Addressed code review

Done.

Done.

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-7347: Addressed code review

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Only reverse when using == operator

Only reverse when using == operator

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

That is what I meant. Why are we even passing this when we know the target is 0?

That is what I meant. Why are we even passing this when we know the target is 0?

Added.

Added.

DEN-7347: Addressed code reviews

    • -34
    • +34
    /firmware/App/Controllers/DGInterface.h
    • -17
    • +1
    /firmware/App/Controllers/PresOccl.c
    • -3
    • +3
    /firmware/App/Controllers/SyringePump.c
    • -6
    • +24
    /firmware/App/Modes/ModePostTreat.c
Changed to return none.

Changed to return none.

Done.

Done.

Done.

Done.

EMPTY_RESERVOIR_VOLUME_ML is defined as 0.

EMPTY_RESERVOIR_VOLUME_ML is defined as 0.

Added.

Added.

I think the variable will be used in future to detect incorrect direction. Sean Nash Please elaborate more.

I think the variable will be used in future to detect incorrect direction.
Sean Nash Please elaborate more.

Done.

Done.

Done.

Done.

The function has been removed.

The function has been removed.