Uploaded image for project: 'IGB'
  1. IGB
  2. IGBF-2253

Implement TwentyThreeAndMeVariationSym using test-first programming

        Details

        • Type: Task
        • Status: Closed (View Workflow)
        • Priority: Major
        • Resolution: Done
        • Affects Version/s: None
        • Fix Version/s: None
        • Labels:
          None
        • Story Points:
          1.5
        • Epic Link:
        • Sprint:
          Spring 3 : 17 Feb to 28 Feb, Spring 4 : 2 Mar to 13 Mar, Spring 5 : 16 Mar to 27 Mar, Spring 6 : 30 Mar to Apr 10, Spring 7 : 13 Apr to 24 Apr, Spring 8 : 24 Apr to 8 May, Spring 8 : 11 May to 25 May, Spring 9 : 25 May to 8 Jun

          Description

          Implement methods in genotyping TwentyThreeAndMeSeqSymmetry class.
          Write javadoc comments and check format by compiling javadocs.

          First:

          • Create unit tests for methods (may require some re-factoring)
          • Add new stub methods as required (in case existing methods should call other methods)

          Next:

          • Implement methods
          • Re-factor class as needed.

          Note: You may need to mock other classes used by TwentyThreeAndMeSeqSymmetry.

            Attachments

              Issue Links

              blocks

              Task - A task that needs to be done. IGBF-2252 Implement TwentyThreeAndMeSymLoader using test-first programming

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                stiwari8 Srishti Tiwari (Inactive) added a comment -

                Hi Dr. Loraine,

                Please review my changes for visualizing 23andme file format:

                https://bitbucket.org/srishtitiwari/integrated-genome-browser/commits/36a86308d9f7e21036a2a50b42383f7538e54a52

                Show
                stiwari8 Srishti Tiwari (Inactive) added a comment - Hi Dr. Loraine, Please review my changes for visualizing 23andme file format: https://bitbucket.org/srishtitiwari/integrated-genome-browser/commits/36a86308d9f7e21036a2a50b42383f7538e54a52
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Thanks! Please see comments on the PR.

                cc: Srishti Tiwari

                Show
                ann.loraine Ann Loraine added a comment - Thanks! Please see comments on the PR. cc: Srishti Tiwari
                Hide
                Permalink
                stiwari8 Srishti Tiwari (Inactive) added a comment -

                Hi Dr. [~aloraine],

                I have made the changes as mentioned in the comments and raised a Pull request. Please review the same:

                https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/781/igbf-2253-igbf-2252-implement/diff

                Show
                stiwari8 Srishti Tiwari (Inactive) added a comment - Hi Dr. [~aloraine] , I have made the changes as mentioned in the comments and raised a Pull request. Please review the same: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/781/igbf-2253-igbf-2252-implement/diff
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Sorry, I should asked about this yesterday:

                There are some lines being deleted - <include>*/.properties</include>

                I have a doubt as to why this required.

                cc: Srishti Tiwari

                Show
                ann.loraine Ann Loraine added a comment - Sorry, I should asked about this yesterday: There are some lines being deleted - <include>* / .properties</include> I have a doubt as to why this required. cc: Srishti Tiwari
                Hide
                Permalink
                stiwari8 Srishti Tiwari (Inactive) added a comment - - edited

                Hi Professor,

                I was seeing this error message in my pom that element <includes> is not allowed here.
                It was also not able to resolve the <include> tag.
                It still shows the same error message.

                Show
                stiwari8 Srishti Tiwari (Inactive) added a comment - - edited Hi Professor, I was seeing this error message in my pom that element <includes> is not allowed here. It was also not able to resolve the <include> tag. It still shows the same error message.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment -

                Ok thanks for the explanation!
                PR is merged.

                cc: Noor Zahara and Srishti Tiwari

                Show
                ann.loraine Ann Loraine added a comment - Ok thanks for the explanation! PR is merged. cc: Noor Zahara and Srishti Tiwari
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment - - edited

                Note: Not yet fully tested. Testing and validation will be first part of next development push.

                Show
                ann.loraine Ann Loraine added a comment - - edited Note: Not yet fully tested. Testing and validation will be first part of next development push.

                  People

                  • Assignee:
                    Unassigned
                    Reporter:
                    ann.loraine Ann Loraine
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: