HD-DEN-7860_HD Dev Bubble Detector Driver

Activity

HD-DEN-7860-1 85

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
    pmontazemi  (deleted user)
    Author & Moderator 1h 56m 28 Same response here.
    qnguyen  (deleted user)
    Reviewer completed
    Reviewer - Complete 1h 38m 28 RESOLVED in CODE WALKTHROUGH.
    Reviewer - Complete 1h 54m 25 I don't see changes to HD fluid leak. DG fluid leak funct...
    Reviewer - Complete 1h 30m 4 RESOLVED in CODE WALKTHROUGH.
    Total   6h 59m 85  
    #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/App/Controllers/BloodLeak.c Changed
    /firmware/App/Controllers/Bubble.c Added 40
    /firmware/App/Controllers/Bubble.h Added
    /firmware/App/Controllers/FluidLeak.c Changed
    /firmware/App/Modes/ModePreTreat.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/SelfTests.c Changed 3
    Open in IDE #permalink
    /firmware/App/Services/AlarmMgmt.h Changed
    /firmware/App/Services/FPGA.c Changed 25
    /firmware/App/Services/FPGA.h Changed
    /firmware/App/Services/SystemComm.c Changed
    Open in IDE #permalink
    /firmware/.../Services/SystemCommMessages.c Changed 8
    Open in IDE #permalink
    /firmware/.../Services/SystemCommMessages.h Changed 9
    Open in IDE #permalink
    /firmware/App/Tasks/TaskPriority.c Changed
    /firmware/source/sys_main.c Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time