DIALIN-DEN-5830_Alarm Design

Activity

DIALIN-DEN-5830-2 13

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 25m 5 moved to tools folder
    plucia  (deleted user)
    Reviewer completed
    Reviewer - Complete 37m 8 RESOLVED
    pmontazemi  (deleted user)
    Reviewer completed
    Reviewer - Complete 2m    
    Total   1h 5m 13  
    #permalink

    Objectives

    Summary:

    • Added API for the message 63: Alarm Condition Cleared
    • Added AlarmFlags
    • Added Alarms Priority enums in the common folder.
    • Updated the CANBus test function test_can0 to be able to be used outside of the SquishQt (ex. Simulator) and moved the common portion in the utils.py



    ----- By Commit
    commit : 00e1e2f - DEN-5833: Implementation

    • Added API for the message 63: Alarm Condition Cleared


    commit : f7da817 - DEN-5833: Implementation

    • Added AlarmFlags


    commit : 2029044 - DEN-5833: Implementation

    • Added Alarms Priority enums in the common folder.
    • Updated the CANBus test function test_can0 to be able to be used out side of the SquishQt (ex. Simulator) and moved the common portion in the utils.py


    commit : a5e1daa

    • Bamboo Commit: Updated the Copyright with the current year and replaced tabs with 4 spaces

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /dialin/common/prs_defs.py Changed
    /dialin/squish/denaliMessages.py Changed
    /dialin/squish/unittests.py Changed 6
    /dialin/squish/utils.py Changed
    /tools/cr_objectives.sh Added
    Open in IDE #permalink
    /cr_objectives.sh Deleted 7
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time