Details
-
Type: Task
-
Status: Closed (View Workflow)
-
Priority: Major
-
Resolution: Done
-
Affects Version/s: None
-
Fix Version/s: None
-
Labels:None
-
Story Points:3
-
Epic Link:
-
Sprint:Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 3: Oct 12 - Oct 23
Description
Situation: Significant changes were made to how the bai sym loader work (IGBF-2329) as well as experimental changes to how bin sizes are calculated and applied to chromosomes (IGBF-2221).
Task: Need to review code changes and refactor code to simplify. For example, it looks like there are now two Java Class files that contain the same code (related to IGBF-2329). Need to determine why there are two classes and if one can be removed.
Need to clean up unnecessary code related to comparing bin sizes to chromosomes (IGBF-2221). After significant testing we determined this could not be accurately implemented. Need to remove any code that refers to it.
Need to double-check code used to determine average size of bins. The average size was originally determined by looping through the data twice. This was fixed in IGBF-2329, but need to make sure that the now unnecessary loop was removed.
Attachments
Issue Links
Activity
Resolution | Done [ 10000 ] | |
Status | Post-merge Testing In Progress [ 10003 ] | Closed [ 6 ] |
Status | Merged Needs Testing [ 10002 ] | Post-merge Testing In Progress [ 10003 ] |
Status | Reviewing Pull Request [ 10303 ] | Merged Needs Testing [ 10002 ] |
Status | Pull Request Submitted [ 10101 ] | Reviewing Pull Request [ 10303 ] |
Status | Ready for Pull Request [ 10304 ] | Pull Request Submitted [ 10101 ] |
Status | First Level Review in Progress [ 10301 ] | Ready for Pull Request [ 10304 ] |
Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
Status | To-Do [ 10305 ] | In Progress [ 3 ] |
Sprint | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 2: 28 Sep - 9 Oct [ 101, 102, 104 ] | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 3: Oct 12 - Oct 23 [ 101, 102, 106 ] |
Sprint | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 1: 14 Sep - 25 Sep [ 101, 102, 103 ] | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 2: 28 Sep - 9 Oct [ 101, 102, 104 ] |
Status | In Progress [ 3 ] | To-Do [ 10305 ] |
Rank | Ranked higher |
Sprint | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep [ 101, 102 ] | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep, Fall 1: 14 Sep - 25 Sep [ 101, 102, 103 ] |
Rank | Ranked higher |
Sprint | Summer 6: 17 Aug - 28 Aug [ 101 ] | Summer 6: 17 Aug - 28 Aug, Summer 7: 31 Aug - 11 Sep [ 101, 102 ] |
Status | To-Do [ 10305 ] | In Progress [ 3 ] |
Status | First Level Review in Progress [ 10301 ] | To-Do [ 10305 ] |
Assignee | Nowlan Freese [ nfreese ] |
Status | Needs 1st Level Review [ 10005 ] | First Level Review in Progress [ 10301 ] |
Assignee | Philip Badzuh [ pbadzuh ] |
Status | In Progress [ 3 ] | Needs 1st Level Review [ 10005 ] |
Status | To-Do [ 10305 ] | In Progress [ 3 ] |
Sprint | Summer 6: 17 Aug - 28 Aug [ 101 ] | |
Assignee | Philip Badzuh [ pbadzuh ] |
Rank | Ranked higher |
Sprint | Summer 5: 29 Jul - 12 Aug [ 100 ] |
Sprint | Summer 4: 14 Jul - 28 Jul [ 99 ] | Summer 5: 29 Jul - 12 Aug [ 100 ] |
Field | Original Value | New Value |
---|---|---|
Epic Link |
|