DD-LEAH-227_FW DD Concentrate Pumps Control

Activity

DD-LEAH-227-1 94

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 52m 47 done.
    Reviewer - Complete 2h 5m 47 Seems like we don't need a switch here. All sensors do th...
    Reviewer - Complete 7m    
    Reviewer - 0% reviewed      
    jpaguio  (deleted user)
    Reviewer - 0% reviewed      
    Total   3h 5m 94  
    #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/ConcentratePumps.c Added 60
    Open in IDE #permalink
    /firmware/.../Controllers/ConcentratePumps.h Added 2
    Open in IDE #permalink
    /firmware/App/Controllers/Valves.c Changed
    Open in IDE #permalink
    /firmware/.../Drivers/ConductivitySensors.c Changed
    Open in IDE #permalink
    /firmware/App/Drivers/InternalADC.c Changed
    Open in IDE #permalink
    /firmware/App/Drivers/InternalADC.h Changed
    Open in IDE #permalink
    /firmware/App/Drivers/PressureSensor.c Changed
    Open in IDE #permalink
    /firmware/App/Drivers/PressureSensor.h Changed
    Open in IDE #permalink
    /firmware/App/Drivers/TemperatureSensors.c Added 16
    Open in IDE #permalink
    /firmware/App/Drivers/TemperatureSensors.h Added
    Open in IDE #permalink
    /firmware/App/Monitors/Conductivity.c Changed
    Open in IDE #permalink
    /firmware/App/Monitors/Pressure.c Changed 4
    Open in IDE #permalink
    /firmware/App/Monitors/Temperature.c Added 6
    Open in IDE #permalink
    /firmware/App/Monitors/Temperature.h Added 2
    Open in IDE #permalink
    /firmware/App/Services/AlarmMgmtSWFaults.h Changed
    Open in IDE #permalink
    /firmware/App/Services/FpgaDD.c Changed
    Open in IDE #permalink
    /firmware/App/Services/FpgaDD.h Changed
    Open in IDE #permalink
    /firmware/App/Services/Messaging.c Changed
    Open in IDE #permalink
    /firmware/App/Tasks/TaskGeneral.c Changed 2
    Open in IDE #permalink
    /firmware/App/Tasks/TaskPriority.c Changed 2
    Open in IDE #permalink
    /firmware/source/sys_main.c Changed
    Open in IDE #permalink
    /AlarmDefs.h Changed
    Open in IDE #permalink
    /MsgDefs.h Changed
    Open in IDE #permalink
    /PersistentAlarm.h Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time