LEAHI-DD-FIRMWARE-LDT-3963_Dialysate Flow Rate - FW Implementation - 1/3:

Activity

LEAHI-DD-FIRMWARE-LDT-3963-1 45

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 3h 58m 10 restored
    Reviewer - 0% reviewed 1m    
    Reviewer - Complete 1h 19m 20 Not sure why Crucible is showing it this way. Let's just ...
    Reviewer - 11% reviewed 23m 3 Should we add a TODO to remove this one as well?
    Reviewer - Complete 2h 45m 11 align the indent.
    Reviewer - 0% reviewed      
    Reviewer - 16% reviewed 30m 1 i see some of the code is for building the code , could y...
    Total   8h 59m 45  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /firmware/.../Controllers/BalancingChamber.c Changed 12
    Open in IDE #permalink
    /firmware/.../Controllers/BalancingChamber.h Changed
    Open in IDE #permalink
    /firmware/.../Controllers/ConcentratePumps.c Changed 3
    Open in IDE #permalink
    /firmware/App/Controllers/DryBiCart.h Changed
    Open in IDE #permalink
    /firmware/App/Controllers/RinsePump.c Changed
    Open in IDE #permalink
    /firmware/App/Controllers/RinsePump.h Changed
    Open in IDE #permalink
    /firmware/App/Drivers/BloodLeakDriver.c Changed 3
    Open in IDE #permalink
    /firmware/.../Drivers/ConductivitySensors.c Changed 5
    Open in IDE #permalink
    /firmware/App/Drivers/ConductivityTeensy.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/ModeFault.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/ModeGenDialysate.c Changed 2
    Open in IDE #permalink
    /firmware/App/Modes/ModeGenDialysate.h Changed
    Open in IDE #permalink
    /firmware/App/Monitors/Level.c Changed 5
    Open in IDE #permalink
    /firmware/App/Monitors/Level.h Changed
    Open in IDE #permalink
    /firmware/App/Monitors/Temperature.c Changed
    Open in IDE #permalink
    /firmware/App/Services/AlarmMgmtDD.c Changed 4
    Open in IDE #permalink
    /firmware/App/Services/FpgaDD.c Changed
    Open in IDE #permalink
    /firmware/App/Services/Messaging.c Changed 2
    Open in IDE #permalink
    /firmware/App/Services/SystemCommDD.c Changed 5
    Open in IDE #permalink
    /firmware/App/DDCommon.h Changed 3
    Open in IDE #permalink
    /AlarmDefs.h Changed 1
    Open in IDE #permalink
    /DDDefs.h Changed
    Open in IDE #permalink
    /MsgDefs.h Changed
    Open in IDE #permalink
    /PIControllers.c Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time