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

            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
            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.
            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
            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.
            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.
            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.

              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: