JAL-2492: Performant lookup of non-positional features


CR-JAL-101 5

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 7m 2 It does look weird, but is just the test harness mimickin...
    Reviewer - Complete 13m 3 this looks incredibly weird - and having to pass sf.varar...
    Total   20m 5  


    Change relevant code to use the SequenceFeatures data store for lookup of non-positional features (replacing scanning of all features).

    Branches in review


    Issues Raised From Comments

    Key Summary State Assignee

    General Comments

    There are no general comments on this review.
    /src/jalview/appletgui/IdPanel.java Changed
    /src/.../features/SequenceFeatures.java Changed 1
    /src/.../features/SequenceFeaturesI.java Changed 2
    /src/jalview/gui/IdPanel.java Changed
    /src/.../io/SequenceAnnotationReport.java Changed
    /test/.../features/SequenceFeaturesTest.java Changed 2
    /test/.../io/SequenceAnnotationReportTest.java Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against