RO-LEAH-262_FW RO Flow Monitor

Activity

RO-LEAH-262-1 20

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 16m 10 Fixed.
    Reviewer - 0% reviewed      
    Reviewer - 98% reviewed 16m    
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - 98% reviewed 31m 10 function body still mentioning pressure.
    Total   1h 3m 20  
    #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.
    /MsgDefs.h Changed
    Open in IDE #permalink
    /firmware/.../Drivers/ConductivitySensors.c Changed 4
    /firmware/.../Drivers/ConductivitySensors.h Changed 2
    /firmware/App/Drivers/FlowSensor.c Added 4
    /firmware/App/Drivers/FlowSensor.h Added
    Open in IDE #permalink
    /firmware/App/Drivers/PressureSensor.c Changed
    /firmware/App/Monitors/Conductivity.c Changed
    /firmware/App/Monitors/Flow.c Added 10
    /firmware/App/Monitors/Flow.h Added
    Open in IDE #permalink
    /firmware/App/Monitors/Pressure.c Changed
    /firmware/App/Services/AlarmMgmtSWFaults.h Changed
    /firmware/App/Services/FpgaRO.c Changed
    /firmware/App/Services/FpgaRO.h Changed
    /firmware/App/Services/Messaging.c Changed
    /firmware/source/sys_main.c Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time