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

Remove repo1 and repo3 from IGB 9.1.8 POM

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
    • Story Points:
      0.25
    • Sprint:
      Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19, Winter 5 Feb 22 - Mar 5

      Attachments

        Issue Links

          Activity

          ann.loraine Ann Loraine created issue -
          ann.loraine Ann Loraine made changes -
          Field Original Value New Value
          Epic Link IGBF-1531 [ 17617 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20 [ 108 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 23 - Dec 4 [ 108, 109 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11 [ 108, 109 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23 [ 108, 109, 110 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ssegu Sai Supreeth Segu (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Assignee Sai Supreeth Segu [ ssegu ]
          ann.loraine Ann Loraine made changes -
          Summary Remove repo1 and repo3 from POM Remove repo1 and repo3 from IGB 9.1.8 POM
          ann.loraine Ann Loraine made changes -
          Labels Beginner
          inaylor Irvin Naylor (Inactive) made changes -
          Assignee Irvin Naylor [ inaylor ]
          inaylor Irvin Naylor (Inactive) made changes -
          Assignee Irvin Naylor [ inaylor ] Ann Loraine [ aloraine ]
          inaylor Irvin Naylor (Inactive) made changes -
          Assignee Ann Loraine [ aloraine ] Irvin Naylor [ inaylor ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          Show
          inaylor Irvin Naylor (Inactive) added a comment - Fork Link: https://bitbucket.org/Inaylor01/integrated-genome-browser/src/IGBF-2596/
          inaylor Irvin Naylor (Inactive) made changes -
          Assignee Irvin Naylor [ inaylor ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Quick request for clarification:

          Please confirm:

          • Does it build in bitbucket?
          Show
          ann.loraine Ann Loraine added a comment - Quick request for clarification: Please confirm: Does it build in bitbucket?
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ]
          ann.loraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Assignee Ann Loraine [ aloraine ] Irvin Naylor [ inaylor ]
          Hide
          inaylor Irvin Naylor (Inactive) added a comment -

          It fails to build, will examine what happened further.

          It says that it cannot find a specific artifact in repo2. Should we remove repo2 along with 1 and 3 as well?

          Show
          inaylor Irvin Naylor (Inactive) added a comment - It fails to build, will examine what happened further. It says that it cannot find a specific artifact in repo2. Should we remove repo2 along with 1 and 3 as well?
          inaylor Irvin Naylor (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          No, you need to retain repo2.

          Show
          ann.loraine Ann Loraine added a comment - No, you need to retain repo2.
          Hide
          inaylor Irvin Naylor (Inactive) added a comment -

          I've run into an issue. So, building the project with repos 1-3 works perfectly however, if I comment out repo1's reference in the pom.xml file (or comment out repo1 and 3's references), I end up having an issue where the project won't build due to repo2's dependencies not being resolved.

          Just wanted to put that out there but will continue trying to figure out the problem

          Show
          inaylor Irvin Naylor (Inactive) added a comment - I've run into an issue. So, building the project with repos 1-3 works perfectly however, if I comment out repo1's reference in the pom.xml file (or comment out repo1 and 3's references), I end up having an issue where the project won't build due to repo2's dependencies not being resolved. Just wanted to put that out there but will continue trying to figure out the problem
          inaylor Irvin Naylor (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23 [ 108, 109, 110 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8 [ 108, 109, 110, 111 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          inaylor Irvin Naylor (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          inaylor Irvin Naylor (Inactive) added a comment -

          Curiously, I've found removing just Repo3 from the pom.xml file allows the project to compile without issue but removing both Repo1 and Repo3 causes this error to come up.

          I made sure to run maven on a clean install, and use the -u command to make sure everything was updated and it still wouldn't budge. My thought is that it has something to do with the bioviz repository itself but I am curious to hear what you all think?

          The error screenshot is attached to the issue.

          Show
          inaylor Irvin Naylor (Inactive) added a comment - Curiously, I've found removing just Repo3 from the pom.xml file allows the project to compile without issue but removing both Repo1 and Repo3 causes this error to come up. I made sure to run maven on a clean install, and use the -u command to make sure everything was updated and it still wouldn't budge. My thought is that it has something to do with the bioviz repository itself but I am curious to hear what you all think? The error screenshot is attached to the issue.
          inaylor Irvin Naylor (Inactive) made changes -
          Attachment Capture_1.PNG [ 15113 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          Hide
          inaylor Irvin Naylor (Inactive) added a comment -

          [~aloraine], Me and Chester Dias just met about the maven error I was running into and we both observed how when Repo 1 and 2 are in the pom.xml file, IGB would build properly. According to the error, there was a certain jar file, jlfgr (more details are in the error) that was missing. We found that it was located in repo1 and that's why it wouldn't build unless 1 was there.

          Chester suggested that we could potentially either move the required jars from repo1 to 2, then get rid of repo1 or we could keep the repository references in a separate maven setting.xml file. I currently have my issue branch where Repos 1 and 2 are in the XML file and wasn't sure if I should submit that for review.

          I wanted to get your take about how we should move forward.

          Show
          inaylor Irvin Naylor (Inactive) added a comment - [~aloraine] , Me and Chester Dias just met about the maven error I was running into and we both observed how when Repo 1 and 2 are in the pom.xml file, IGB would build properly. According to the error, there was a certain jar file, jlfgr (more details are in the error) that was missing. We found that it was located in repo1 and that's why it wouldn't build unless 1 was there. Chester suggested that we could potentially either move the required jars from repo1 to 2, then get rid of repo1 or we could keep the repository references in a separate maven setting.xml file. I currently have my issue branch where Repos 1 and 2 are in the XML file and wasn't sure if I should submit that for review. I wanted to get your take about how we should move forward.
          Hide
          cdias1 Chester Dias (Inactive) added a comment -

          [~aloraine] Please let us know if there is any specific reason for having multiple repos? Can we move all the required artifacts into one repo?

          Show
          cdias1 Chester Dias (Inactive) added a comment - [~aloraine] Please let us know if there is any specific reason for having multiple repos? Can we move all the required artifacts into one repo?
          Hide
          ann.loraine Ann Loraine added a comment -

          Yes we can, but it's hard to do because there are a lot of details to consider.

          Show
          ann.loraine Ann Loraine added a comment - Yes we can, but it's hard to do because there are a lot of details to consider.
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8 [ 108, 109, 110, 111 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22 [ 108, 109, 110, 111, 112 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          inaylor Irvin Naylor (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
          Hide
          inaylor Irvin Naylor (Inactive) added a comment - - edited
          Show
          inaylor Irvin Naylor (Inactive) added a comment - - edited Pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/859/igbf-2596-remove-repo1-and-3-from-918 Commit where repo3 is removed, leaving only 1 and 2
          inaylor Irvin Naylor (Inactive) made changes -
          Status To-Do [ 10305 ] Pull Request Submitted [ 10101 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Request for functional review prior to merging:

          • Please recruit one person on Windows and another person on Mac.

          Test on each platform as follows:

          • Install IGB
          • Review logging messages - look for any "class not found" errors
          • Open Arabidopsis genome and load some data from "Reads" and "Graphs" folders, again observing the console for possible errors
          Show
          ann.loraine Ann Loraine added a comment - Request for functional review prior to merging: Please recruit one person on Windows and another person on Mac. Test on each platform as follows: Install IGB Review logging messages - look for any "class not found" errors Open Arabidopsis genome and load some data from "Reads" and "Graphs" folders, again observing the console for possible errors
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22 [ 108, 109, 110, 111, 112 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5 [ 108, 109, 110, 111, 112, 113 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          inaylor Irvin Naylor (Inactive) made changes -
          Assignee Irvin Naylor [ inaylor ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5 [ 108, 109, 110, 111, 112, 113 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19 [ 108, 109, 110, 111, 112, 113, 114 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Sprint Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19 [ 108, 109, 110, 111, 112, 113, 114 ] Fall 5 Nov 9 - Nov 20, Fall 6 Nov 30 - Dec 11, Fall 7 Dec 14 - Dec 23, Winter 1 Dec 28 - Jan 8, Winter 2 Jan 11 - Jan 22, Winter 3 Jan 25 - Feb 5, Winter 4 Feb 8 - Feb 19, Winter 5 Feb 22 - Mar 5 [ 108, 109, 110, 111, 112, 113, 114, 115 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Logan Weidenhammer] - Could you please test Irvin's branch installer as per the above procedure?

          Show
          ann.loraine Ann Loraine added a comment - Logan Weidenhammer ] - Could you please test Irvin's branch installer as per the above procedure?
          ann.loraine Ann Loraine made changes -
          Assignee Rachel Weidenhammer [ rweidenh ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          Hide
          rweidenh Logan Weidenhammer (Inactive) added a comment -

          Functional Review on Windows:

          Installed IGB from Irvin's Fork
          Reviewed the Log and did not find any 'class not found' errors.
          Opened Arabidopsis genome
          Loaded data for a read alignment, and a scaled coverage graph
          Reviewed the Log again and did not find a 'class not found' error.

          It looks like we still need a MacOS functional review.

          Show
          rweidenh Logan Weidenhammer (Inactive) added a comment - Functional Review on Windows: Installed IGB from Irvin's Fork Reviewed the Log and did not find any 'class not found' errors. Opened Arabidopsis genome Loaded data for a read alignment, and a scaled coverage graph Reviewed the Log again and did not find a 'class not found' error. It looks like we still need a MacOS functional review.
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status First Level Review in Progress [ 10301 ] Needs 1st Level Review [ 10005 ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Assignee Rachel Weidenhammer [ rweidenh ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Thanks Logan Weidenhammer!
          Probably a Mac review won't be needed. So would recommend moving ahead with a PR.

          Show
          ann.loraine Ann Loraine added a comment - Thanks Logan Weidenhammer ! Probably a Mac review won't be needed. So would recommend moving ahead with a PR.
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          rweidenh Logan Weidenhammer (Inactive) made changes -
          Assignee Irvin Naylor [ inaylor ]
          ann.loraine Ann Loraine made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ann.loraine Ann Loraine made changes -
          Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          ann.loraine Ann Loraine made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          ann.loraine Ann Loraine made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          ann.loraine Ann Loraine made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Change of plan:

          • Do not remove these from the POM at this time.
          • We will need to check that this is feasible after returning DAS1 support.
          • Moving to Done.
          Show
          ann.loraine Ann Loraine added a comment - Change of plan: Do not remove these from the POM at this time. We will need to check that this is feasible after returning DAS1 support. Moving to Done.
          ann.loraine Ann Loraine made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          ann.loraine Ann Loraine made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          ann.loraine Ann Loraine made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          ann.loraine Ann Loraine made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
          inaylor Irvin Naylor (Inactive) made changes -
          Assignee Irvin Naylor [ inaylor ]
          ann.loraine Ann Loraine made changes -
          Assignee Irvin Naylor [ inaylor ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Note for Irvin Naylor - leaving this assigned to you reflecting the work you have done on it.

          Show
          ann.loraine Ann Loraine added a comment - Note for Irvin Naylor - leaving this assigned to you reflecting the work you have done on it.
          ann.loraine Ann Loraine made changes -
          Link This issue relates to IGBF-3382 [ IGBF-3382 ]

            People

            • Assignee:
              inaylor Irvin Naylor (Inactive)
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: