hdfirmware

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
DEN-14001: Addressed code review comments.

Fixed.

Fixed.

Added.

Added.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Fixed.

Fixed.

Agree. Change "_SECOND" to "_IN_TREATMENT"

Agree. Change "_SECOND" to "_IN_TREATMENT"

This should be the first if and the one above should just be an else (no if).

This should be the first if and the one above should just be an else (no if).

Remove extra space after static.

Remove extra space after static.

If not used should be removed.

If not used should be removed.

DEN-13834 addressed the code review comments and initialized some variables per CppCheck

Added them to the doxygen comments.

Added them to the doxygen comments.

Update function header. dialysateInPumpRotorCounter.data, dipStopAtHomePosition also output.

Update function header. dialysateInPumpRotorCounter.data, dipStopAtHomePosition also output.

RESOLVED IN CODE WALKTHROUGH

RESOLVED IN CODE WALKTHROUGH

Are bloodLeakEmbModeHasZeroBeenRqustd and bloodLeakSelfTestStatus both global?

Are bloodLeakEmbModeHasZeroBeenRqustd and bloodLeakSelfTestStatus both global?

HD-DEN-14150_HD Dialysate Flow Estimation
HD-DEN-14150_HD Dialysate Flow Estimation
DEN-13801: Dry Self changes based on feedback.

Done.

Done.

DEN-13834 addressed the code review comments

    • -228
    • +27
    /firmware/App/Services/FPGA.c
Removed the "no event" event.

Removed the "no event" event.

Removed it.

Removed it.

Done.

Done.

Done.

Done.

Done.

Done.

It is uncommented.

It is uncommented.

Done.

Done.

This is an old code and it has been removed.

This is an old code and it has been removed.

This is an old code and it has been removed.

This is an old code and it has been removed.

This is an old code and it has been removed.

This is an old code and it has been removed.

DEN-13801: reworked dry self test tolerances to be more in line with data.