LEAHI-DD-FIRMWARE-LDT-2936_IOFP - Update M12 and Boost Pump logic in FW for...

Activity

LEAHI-DD-FIRMWARE-LDT-2936-1 86

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 5m 43 Resolved
    Reviewer - Complete 16m    
    Reviewer - 0% reviewed      
    Reviewer - 57% reviewed 5m    
    Reviewer - Complete 1h 40m 31 Change comment from 3 to 5 minutes to match code.
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - 100% reviewed 3h 12 Small letter used
    Reviewer - 87% reviewed 13m    
    Reviewer - 0% reviewed      
    Reviewer - Complete 1h 14m    
    Total   9h 36m 86  
    #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/PermeateTank.c Changed
    Open in IDE #permalink
    /firmware/App/Controllers/ROPump.c Changed 10
    Open in IDE #permalink
    /firmware/App/Drivers/FluidPump.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/FPModes/FPModeFault.c Changed
    Open in IDE #permalink
    /firmware/App/.../FPModes/FPModeStandby.c Changed
    Open in IDE #permalink
    /firmware/App/.../FPModes/FPOperationModes.c Changed
    Open in IDE #permalink
    /firmware/App/.../FPModes/FlushConcentrate.c Changed 9
    Open in IDE #permalink
    /firmware/App/Modes/FPModes/FlushFilter.c Changed 2
    Open in IDE #permalink
    /firmware/App/.../FPModes/FlushPermeate.c Changed 4
    Open in IDE #permalink
    /firmware/App/.../FPModes/ModeGenPermeate.c Changed 17
    Open in IDE #permalink
    /firmware/App/.../FPModes/ModeGenPermeate.h Changed
    Open in IDE #permalink
    /firmware/.../FPModes/ModeGenPermeateDefeatured.c Changed
    Open in IDE #permalink
    /firmware/App/.../FPModes/ModePreGenPermeate.c Changed 6
    Open in IDE #permalink
    /firmware/.../FPModes/ModePreGenPermeateDefeatured.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/ModeStandby.c Changed
    Open in IDE #permalink
    /firmware/App/Monitors/Conductivity.c Changed 22
    Open in IDE #permalink
    /firmware/App/Monitors/Conductivity.h Changed 8
    Open in IDE #permalink
    /firmware/.../Monitors/WaterQualityMonitor.c Changed 2
    Open in IDE #permalink
    /firmware/.../Monitors/WaterQualityMonitor.h Changed
    Open in IDE #permalink
    /firmware/App/Services/Messaging.c Changed
    Open in IDE #permalink
    /firmware/App/Services/TDInterface.c Changed
    Open in IDE #permalink
    /AlarmDefs.h Changed 2
    Open in IDE #permalink
    /FPDefs.h Changed
    Open in IDE #permalink
    /MsgDefs.h Changed 4
    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