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

Fix loading bookmark from Galaxy when genome from UCSC

    Details

    • Type: Improvement
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: 10.1.0
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23, Spring 10

      Description

      Situation: Many genomes in IGB are provided by UCSC DAS, a service that is similar to Quickload. Galaxy is a website that provides analysis tools for genomes. The data produced by Galaxy can then be viewed in IGB by clicking a link in Galaxy. The link works through the BookmarkUnibrowControlServlet.java class to parse out the genome and other information.

      However, there is an edge case where if the genome is provided by UCSC DAS in IGB (as opposed to IGB Quickload) and the user has not first selected the correct genome, clicking the link in Galaxy will cause an error (see comment). This is because seqid is null. The seqid should be set to a chromosome/scaffold/contig of the genome. My guess is that because the genome is provided by UCSC DAS, which behaves differently than Quickload, the current methods to retrieve the seqid for the genome are returning null.

      Task: Implement a fix.

        Attachments

          Issue Links

            Activity

            nfreese Nowlan Freese created issue -
            nfreese Nowlan Freese made changes -
            Field Original Value New Value
            Epic Link IGBF-1880 [ 17970 ]
            nfreese Nowlan Freese made changes -
            Link This issue relates to IGBF-1879 [ IGBF-1879 ]
            Hide
            nfreese Nowlan Freese added a comment -

            No seqid: The bookmark did not specify a valid seqid: specified 'null'
            -------------------------------------------------------

            14:22:20.856 ERROR c.a.i.b.BookmarkUnibrowControlServlet - Error while loading bookmark.
            java.lang.NullPointerException: null
            at com.affymetrix.igb.bookmarks.BookmarkUnibrowControlServlet$1.runInBackground(BookmarkUnibrowControlServlet.java:224) ~[classes/:na]
            at com.affymetrix.genometry.thread.CThreadWorker.doInBackground(CThreadWorker.java:73) [genometry-9.1.8.jar:na]
            at javax.swing.SwingWorker$1.call(SwingWorker.java:295) [na:1.8.0_191]
            at java.util.concurrent.FutureTask.run(FutureTask.java:266) [na:1.8.0_191]
            at javax.swing.SwingWorker.run(SwingWorker.java:334) [na:1.8.0_191]
            at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [na:1.8.0_191]
            at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [na:1.8.0_191]
            at java.lang.Thread.run(Thread.java:748) [na:1.8.0_191]

            Show
            nfreese Nowlan Freese added a comment - No seqid: The bookmark did not specify a valid seqid: specified 'null' ------------------------------------------------------- 14:22:20.856 ERROR c.a.i.b.BookmarkUnibrowControlServlet - Error while loading bookmark. java.lang.NullPointerException: null at com.affymetrix.igb.bookmarks.BookmarkUnibrowControlServlet$1.runInBackground(BookmarkUnibrowControlServlet.java:224) ~ [classes/:na] at com.affymetrix.genometry.thread.CThreadWorker.doInBackground(CThreadWorker.java:73) [genometry-9.1.8.jar:na] at javax.swing.SwingWorker$1.call(SwingWorker.java:295) [na:1.8.0_191] at java.util.concurrent.FutureTask.run(FutureTask.java:266) [na:1.8.0_191] at javax.swing.SwingWorker.run(SwingWorker.java:334) [na:1.8.0_191] at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [na:1.8.0_191] at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [na:1.8.0_191] at java.lang.Thread.run(Thread.java:748) [na:1.8.0_191]
            Show
            nfreese Nowlan Freese added a comment - - edited To replicate: In IGB, navigate to File > Preferences... and then select the Data Sources tab Disable all data sources except for UCSC DAS Close and reopen IGB (do not reset preferences to default) In a web browser, paste the following link: https://bioviz.org/galaxy.html?version=mm10&feature_url_0=https://usegalaxy.org/display_application/bbd44e69cb8906b5831e0a45935827e7/igb_bam/View/None/data/RNA_seq_Sox9_Control__Tophat2_accepted_hits_.bam&sym_name_0=RNA_seq_Sox9_Control__Tophat2_accepted_hits_&sym_method_0=https://usegalaxy.org/display_application/bbd44e69cb8906b5831e0a45935827e7/igb_bam/View/None/data/RNA_seq_Sox9_Control__Tophat2_accepted_hits_.bam&query_url=https://usegalaxy.org/display_application/bbd44e69cb8906b5831e0a45935827e7/igb_bam/View/None/data/RNA_seq_Sox9_Control__Tophat2_accepted_hits_.bam&server_url=galaxy
            ann.loraine Ann Loraine made changes -
            Sprint Spring 6 2021 May 31 - June 11 [ 122 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25 [ 122, 123 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Sprint Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25 [ 122, 123 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9 [ 122, 123, 124 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Sprint Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9 [ 122, 123, 124 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23 [ 122, 123, 124, 125 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Sprint Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23 [ 122, 123, 124, 125 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23, Summer 4 2021 Aug 2 - Aug 13 [ 122, 123, 124, 125, 126 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            nfreese Nowlan Freese made changes -
            Sprint Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23, Summer 4 2021 Aug 2 - Aug 13 [ 122, 123, 124, 125, 126 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23 [ 122, 123, 124, 125 ]
            nfreese Nowlan Freese made changes -
            Sprint Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23 [ 122, 123, 124, 125 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23, Summer 1 [ 122, 123, 124, 125, 195 ]
            jsirigin Jaya Sravani Sirigineedi made changes -
            Assignee Jaya Sravani Sirigineedi [ jsirigin ]
            jsirigin Jaya Sravani Sirigineedi made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            nfreese Nowlan Freese made changes -
            Sprint Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23, Summer 1 [ 122, 123, 124, 125, 195 ] Spring 6 2021 May 31 - June 11, Summer 1 2021 Jun 14 - Jun 25, Summer 2 2021 Jun 28 - Jul 9, Summer 3 2021 Jul 12 - Jul 23, Spring 10 [ 122, 123, 124, 125, 194 ]
            jsirigin Jaya Sravani Sirigineedi made changes -
            Hide
            jsirigin Jaya Sravani Sirigineedi added a comment -

            Fixed the issue with the loading when the genome is not present in IGB Quickload, had to change the genome version name that was sent to GeneralLoadView initVersion method, before a wrong version name was sent because of which if the data is no there in IGB Quickload it wasn't able to load the chromosomes using that version name for the remaining providers. Also, had to add the autoload functionality for UCSC to load the data which is done as part of this ticket: https://jira.bioviz.org/browse/IGBF-3633. Updated code is present at the branch: https://bitbucket.org/jaya-sravani/integrated-genome-browser/branch/IGBF-2862. Testing is completed, working as expected.

            Show
            jsirigin Jaya Sravani Sirigineedi added a comment - Fixed the issue with the loading when the genome is not present in IGB Quickload, had to change the genome version name that was sent to GeneralLoadView initVersion method, before a wrong version name was sent because of which if the data is no there in IGB Quickload it wasn't able to load the chromosomes using that version name for the remaining providers. Also, had to add the autoload functionality for UCSC to load the data which is done as part of this ticket: https://jira.bioviz.org/browse/IGBF-3633 . Updated code is present at the branch: https://bitbucket.org/jaya-sravani/integrated-genome-browser/branch/IGBF-2862 . Testing is completed, working as expected.
            jsirigin Jaya Sravani Sirigineedi made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            jsirigin Jaya Sravani Sirigineedi made changes -
            Assignee Jaya Sravani Sirigineedi [ jsirigin ]
            Hide
            nfreese Nowlan Freese added a comment - - edited

            Tested Sravani's branch on Mac:
            BioViz Connect - worked
            Public trackhubs - worked
            Galaxy - worked

            I was able to load data without seeing the seqid error when genome was only available from UCSC REST.

            Ready for pull request

            Show
            nfreese Nowlan Freese added a comment - - edited Tested Sravani's branch on Mac: BioViz Connect - worked Public trackhubs - worked Galaxy - worked I was able to load data without seeing the seqid error when genome was only available from UCSC REST. Ready for pull request
            nfreese Nowlan Freese made changes -
            Assignee Jaya Sravani Sirigineedi [ jsirigin ]
            nfreese Nowlan Freese made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            nfreese Nowlan Freese made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            Hide
            jsirigin Jaya Sravani Sirigineedi added a comment -

            Will raise the pull request once this one is merged as both have a same commit: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1013

            Show
            jsirigin Jaya Sravani Sirigineedi added a comment - Will raise the pull request once this one is merged as both have a same commit: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1013
            Hide
            jsirigin Jaya Sravani Sirigineedi added a comment -

            Raised the pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1014. Please review and let me know if there are any issues.

            Show
            jsirigin Jaya Sravani Sirigineedi added a comment - Raised the pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1014 . Please review and let me know if there are any issues.
            jsirigin Jaya Sravani Sirigineedi made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            jsirigin Jaya Sravani Sirigineedi made changes -
            Assignee Jaya Sravani Sirigineedi [ jsirigin ]
            ann.loraine Ann Loraine made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            ann.loraine Ann Loraine made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged into "main" branch as per usual. Installers are built and deployed to bioviz.org "early access" section.

            Ready for testing.

            Show
            ann.loraine Ann Loraine added a comment - PR is merged into "main" branch as per usual. Installers are built and deployed to bioviz.org "early access" section. Ready for testing.
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            nfreese Nowlan Freese made changes -
            Story Points 3 1
            nfreese Nowlan Freese made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            Hide
            nfreese Nowlan Freese added a comment -

            Tested using main branch installer on Mac.

            With only the UCSC REST data provider activated, I was able to load genomes from public trackhubs and Galaxy without seeing the seqid error.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - Tested using main branch installer on Mac. With only the UCSC REST data provider activated, I was able to load genomes from public trackhubs and Galaxy without seeing the seqid error. Closing ticket.
            nfreese Nowlan Freese made changes -
            Assignee Jaya Sravani Sirigineedi [ jsirigin ]
            nfreese Nowlan Freese made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]
            nfreese Nowlan Freese made changes -
            Fix Version/s 10.1.0 [ 11000 ]

              People

              • Assignee:
                jsirigin Jaya Sravani Sirigineedi
                Reporter:
                nfreese Nowlan Freese
              • Votes:
                0 Vote for this issue
                Watchers:
                Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: