CR-JAL-101 5
- Details
- Objectives
- General Comments 0
- Unresolved
- Resolved
- Number of files included: 7
-
jalview
0
-
Folder
src/jalview
0
-
Folder
appletgui
0
- File IdPanel.java 0 Remove
-
Folder
datamodel/features
0
- File SequenceFeatures.java 1 Remove
- File SequenceFeaturesI.java 2 Remove
-
Folder
gui
0
- File IdPanel.java 0 Remove
-
Folder
io
0
- File SequenceAnnotationReport.java 0 Remove
-
Folder
appletgui
0
-
Folder
test/jalview
0
-
Folder
datamodel/features
0
- File SequenceFeaturesTest.java 2 Remove
-
Folder
io
0
- File SequenceAnnotationReportTest.java 0 Remove
-
Folder
datamodel/features
0
-
Folder
src/jalview
0
-
Filter
- Only show me content:
- Unfiltered files: dynamically added content
- Filtered files: dynamically added content
- Clear filters
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 | 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 |
-
Linked Issue:
JAL-2492
-
#permalink
Objectives
Change relevant code to use the SequenceFeatures data store for lookup of non-positional features (replacing scanning of all features).
Branches in review
Repository | Branch to review | Branched from |
---|
/src/jalview/appletgui/IdPanel.java
Changed
0
(0)
/src/.../features/SequenceFeatures.java
Changed
1
(0)
/src/.../features/SequenceFeaturesI.java
Changed
2
(0)
/src/jalview/gui/IdPanel.java
Changed
0
(0)
/src/.../io/SequenceAnnotationReport.java
Changed
0
(0)
/test/.../features/SequenceFeaturesTest.java
Changed
2
(0)
/test/.../io/SequenceAnnotationReportTest.java
Changed
0
(0)