JAL-2386: Change Colour (All Groups) applies PID threshold but not Conservation


CR-JAL-28 0

Keyboard shortcuts  
  • Summarize the review outcomes (optional)


    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author 0m    
    Reviewer - Complete 6m    
    Total   6m 0  


    Behaviour of Change Colour with Apply to All Groups selected is inconsistent:

    • PID threshold is applied to all groups along with the selected colour scheme
    • Consensus slider value is not

    For example,

    • load uniref50.fa and colour by helix propensity
    • create a subgroup and colour by turn propensity
    • apply a PID threshold to the alignment colour scheme
    • select Colour | Apply Colour to All Groups and change to Hydrophobicity
      -> both colour and PID threshold are also applied to the sub-group
      Compare the same procedure but with By Conservation instead of PID:
      -> the Conservation threshold is not applied to the sub-group

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/jalview/appletgui/AlignFrame.java Changed
    /src/jalview/gui/AlignFrame.java Changed
    /src/jalview/gui/PopupMenu.java Changed
    /src/jalview/gui/SliderPanel.java Changed
    /src/jalview/jbgui/GAlignFrame.java Changed
    /src/.../viewmodel/AlignmentViewport.java Changed
    /test/jalview/gui/AlignFrameTest.java Changed

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against