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

Add CRAM to list of supported files for BioViz Connect

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      Situation: BioViz Connect uses a hardcoded list of IGB accepted file format extensions in order to correctly populate the View in IGB button. As we have now added support for cram, we need to update the list of file formats in BioViz Connect.

      Task: Update BioViz Connect so that CRAM is an accepted file format. Note that as previous versions of IGB do not support the cram format, we will need to add some additional logic to BioViz Connect to check the version number of IGB and limit the View in IGB button for cram to newer versions.

        Attachments

          Issue Links

            Activity

            Hide
            pkulzer Paige Kulzer added a comment -

            Here is some additional information on the CRAM file format if anyone would like to better familiarize themselves: https://academic.oup.com/bioinformatics/article/38/6/1497/6499262

            Show
            pkulzer Paige Kulzer added a comment - Here is some additional information on the CRAM file format if anyone would like to better familiarize themselves: https://academic.oup.com/bioinformatics/article/38/6/1497/6499262
            Hide
            karthik Karthik Raveendran added a comment -

            Added cram to IGB supported file format. See commit

            Show
            karthik Karthik Raveendran added a comment - Added cram to IGB supported file format. See commit
            Hide
            nfreese Nowlan Freese added a comment -

            Karthik Raveendran - how do we want to tackle version checking for cram?

            Option 1: Do nothing, any user who is using an older version of IGB will see the View in IGB button for a CRAM file, but when clicked it will throw an error in IGB.
            Option 2: Update logic in BioViz Connect to check the IGB version (major/minor/patch) and anything less than 10.0.0 will not show the View in IGB button for CRAM files.
            Option 3: Update logic in BioViz Connect to check the IGB version (major/minor/patch) and anything less than 10.0.0 will show a modal informing the user "To view a CRAM file in IGB, please update your IGB to the most recent version".

            I think we are checking IGB version in cyverse.js and there is a popup_modals.html warning for older versions of IGB.

            Show
            nfreese Nowlan Freese added a comment - Karthik Raveendran - how do we want to tackle version checking for cram? Option 1: Do nothing, any user who is using an older version of IGB will see the View in IGB button for a CRAM file, but when clicked it will throw an error in IGB. Option 2: Update logic in BioViz Connect to check the IGB version (major/minor/patch) and anything less than 10.0.0 will not show the View in IGB button for CRAM files. Option 3: Update logic in BioViz Connect to check the IGB version (major/minor/patch) and anything less than 10.0.0 will show a modal informing the user "To view a CRAM file in IGB, please update your IGB to the most recent version". I think we are checking IGB version in cyverse.js and there is a popup_modals.html warning for older versions of IGB.
            Hide
            karthik Karthik Raveendran added a comment -

            Implemented Option 3 from the prev comment by Dr. Nowlan Freese. See commit

            Show
            karthik Karthik Raveendran added a comment - Implemented Option 3 from the prev comment by Dr. Nowlan Freese . See commit
            Show
            karthik Karthik Raveendran added a comment - PR Submitted. https://bitbucket.org/nfreese/bioviz-connect/pull-requests/291
            Hide
            ann.loraine Ann Loraine added a comment -

            To deploy testing and new versions using code in bioviz connect playbooks repository.

            AL request: Karthik Raveendran to take over deploying new versions of bioviz connect code, using playbooks.

            Show
            ann.loraine Ann Loraine added a comment - To deploy testing and new versions using code in bioviz connect playbooks repository. AL request: Karthik Raveendran to take over deploying new versions of bioviz connect code, using playbooks.
            Hide
            nfreese Nowlan Freese added a comment - - edited

            Merged and deployed to connect.bioviz.org/

            Deployed following instructions from bioviz-connect-playbooks repo with:

            ansible-playbook -i inventory.ini setup.yml

            Once the playbook was completed I had to hit the SyncAppDataToBioViz endpoint to see the change:

            https://connect.bioviz.org/SyncAppDataToBioViz/?accesstoken=<validKeyCloakAccessToken>
            

            I first tested the changes on bvctest7 (Nowlan has access to start/stop the instance).

            Note that you will need to proper crt files and key. You will also need to have a properly filled out inventory.ini and group_vars > setup_vars.yml files. I also specify my repo in the roles > clone > defaults > main.yml file:

            repo: https://bitbucket.org/nfreese/bioviz-connect.git
            Show
            nfreese Nowlan Freese added a comment - - edited Merged and deployed to connect.bioviz.org/ Deployed following instructions from bioviz-connect-playbooks repo with: ansible-playbook -i inventory.ini setup.yml Once the playbook was completed I had to hit the SyncAppDataToBioViz endpoint to see the change: https: //connect.bioviz.org/SyncAppDataToBioViz/?accesstoken=<validKeyCloakAccessToken> I first tested the changes on bvctest7 (Nowlan has access to start/stop the instance). Note that you will need to proper crt files and key. You will also need to have a properly filled out inventory.ini and group_vars > setup_vars.yml files. I also specify my repo in the roles > clone > defaults > main.yml file: repo: https: //bitbucket.org/nfreese/bioviz-connect.git
            Hide
            nfreese Nowlan Freese added a comment -

            I uploaded a folder with matching cram/bam/sam files. I was able to modify the metadata to specify the Arabidopsis genome for the CRAM file. Once the CRAM file and its index were made public I was able to click View in IGB and view the files in IGB. If I tried to load a CRAM file into a version of IGB less than 10.0.0 I saw a warning from BioViz Connect that I should upgrade my version of IGB.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - I uploaded a folder with matching cram/bam/sam files. I was able to modify the metadata to specify the Arabidopsis genome for the CRAM file. Once the CRAM file and its index were made public I was able to click View in IGB and view the files in IGB. If I tried to load a CRAM file into a version of IGB less than 10.0.0 I saw a warning from BioViz Connect that I should upgrade my version of IGB. Closing ticket.

              People

              • Assignee:
                karthik Karthik Raveendran
                Reporter:
                nfreese Nowlan Freese
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: