firmware

Clone Tools
  • last updated a few minutes ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-5963 addressed code review comments

    • -1
    • +5
    ./App/Controllers/TemperatureSensors.c
    • -3
    • +11
    ./App/Services/SystemCommMessages.c
Nice catch. Done.

Nice catch. Done.

Done.

Done.

This section was for debugging only. I removed it.

This section was for debugging only. I removed it.

[~qnguyen] Could you please address?

[~qnguyen] Could you please address?

I spelled it completely.

I spelled it completely.

The small and main primary duty cycles are on different PWM channels. They cannot be controlled with one function.

The small and main primary duty cycles are on different PWM channels. They cannot be controlled with one function.

This struct is the structure that is used to set the dialysate target temperature using the CAN bus. I do not think we can use enums for this purpose.

This struct is the structure that is used to set the dialysate target temperature using the CAN bus. I do not think we can use enums for this purpose.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

Done.

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.

Moved to common DGDefs.h (in staging branch).

Moved to common DGDefs.h (in staging branch).

DEN-7568 added timeout conditional statements for mode flush

DEN-7568 developed and tested the flush mode

    • -196
    • +138
    ./App/Controllers/ROPump.c
DEN-7568 worked on mode flush

    • -2
    • +2
    ./App/Controllers/TemperatureSensors.c
    • -350
    • +372
    ./App/Services/Reservoirs.c
    • -1
    • +71
    ./App/Services/SystemCommMessages.c
DEN-7802: Updated concentrate pump driver

    • -1
    • +2
    ./App/Controllers/TemperatureSensors.c
Comments shall start with capital letter.

Comments shall start with capital letter.

Replace a with a more meaningful variable name.

Replace a with a more meaningful variable name.

Align comment.

Align comment.

Align comment.

Align comment.

Add TODO or delete.

Add TODO or delete.

Replace logic with a temp parameter (call it whatever you like) and then use that parameter in the function call. This will make the code a lot more legible.

Replace logic with a temp parameter (call it whatever you like) and then use that parameter in the function call. This will make the code a lot more legible.