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

Add SNP File Converter icon to IGB

    Details

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

      Description

      The SNP File Converter icon that is currently being used in the IGB App Manager should be added to the Tools menu in IGB. The icon can be found on Bitbucket: https://bitbucket.org/lorainelab/snp-file-converter/src/master/src/main/resources/images/23andMe-snp-converter-logo.png

        Attachments

          Issue Links

            Activity

            pkulzer Paige Kulzer created issue -
            pkulzer Paige Kulzer made changes -
            Field Original Value New Value
            Epic Link IGBF-1765 [ 17855 ]
            pkulzer Paige Kulzer made changes -
            Link This issue relates to IGBF-3851 [ IGBF-3851 ]
            Hide
            ann.loraine Ann Loraine added a comment -

            Doing this will likely require editing the SNP Converter code itself, not IGB.
            We did this for ProtAnnot at some point, so we can look through the commits in the ProtAnnot repository for ideas on how to make this happen for the SNP Converter App.

            Question: Why is it called "SNP Converter" ?

            It sounds like it transforms SNPs into something else. I don't think this is what it does, though. It converts a file format into some other file format - is that right? If yes, I think we should try to think of a name that is more specific about what the App does.

            How about: "23-and-Me file type converter"

            Possible cool use case:

            • Open a 23-and-Me data file for a person
            • Open a genome sequence data file for the same person
            • Compare the genotype calls - do they agree ?

            This could be a really cool demonstration of IGB capabilities!

            Show
            ann.loraine Ann Loraine added a comment - Doing this will likely require editing the SNP Converter code itself, not IGB. We did this for ProtAnnot at some point, so we can look through the commits in the ProtAnnot repository for ideas on how to make this happen for the SNP Converter App. Question: Why is it called "SNP Converter" ? It sounds like it transforms SNPs into something else. I don't think this is what it does, though. It converts a file format into some other file format - is that right? If yes, I think we should try to think of a name that is more specific about what the App does. How about: "23-and-Me file type converter" Possible cool use case: Open a 23-and-Me data file for a person Open a genome sequence data file for the same person Compare the genotype calls - do they agree ? This could be a really cool demonstration of IGB capabilities!
            ann.loraine Ann Loraine made changes -
            Sprint Summer 6 [ 200 ] Summer 6, Summer 7 [ 200, 201 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            pkulzer Paige Kulzer made changes -
            Assignee Dylan Marrotte [ dmarrott ]
            pkulzer Paige Kulzer made changes -
            Assignee Paige Kulzer [ pkulzer ]
            nfreese Nowlan Freese made changes -
            Link This issue relates to IGBF-3661 [ IGBF-3661 ]
            nfreese Nowlan Freese made changes -
            Link This issue relates to IGBF-3551 [ IGBF-3551 ]
            Hide
            nfreese Nowlan Freese added a comment -

            I don't think we need to add a right-click option for the converter option, so please look at IGBF-3551 for the details of how to add the icon to the Tools menu.

            Show
            nfreese Nowlan Freese added a comment - I don't think we need to add a right-click option for the converter option, so please look at IGBF-3551 for the details of how to add the icon to the Tools menu.
            nfreese Nowlan Freese made changes -
            Link This issue blocks IGBF-3858 [ IGBF-3858 ]
            pkulzer Paige Kulzer made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            pkulzer Paige Kulzer added a comment -

            The app's icon has been added to the tools menu. As for Ann's comment, maybe we can create a separate "investigate" ticket regarding the app's name since this ticket's scope only includes the addition of that icon.

            Branch: https://bitbucket.org/pkulzer-lorainelab/snp-file-converter/branch/IGBF-3850?dest=lorainelab%2Fsnp-file-converter%3Amaster

            Show
            pkulzer Paige Kulzer added a comment - The app's icon has been added to the tools menu. As for Ann's comment, maybe we can create a separate "investigate" ticket regarding the app's name since this ticket's scope only includes the addition of that icon. Branch: https://bitbucket.org/pkulzer-lorainelab/snp-file-converter/branch/IGBF-3850?dest=lorainelab%2Fsnp-file-converter%3Amaster
            pkulzer Paige Kulzer made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pkulzer Paige Kulzer made changes -
            Assignee Paige Kulzer [ pkulzer ] Nowlan Freese [ nfreese ]
            nfreese Nowlan Freese made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            Hide
            nfreese Nowlan Freese added a comment -

            Tested Paige's branch on Mac.

            App's icon now appears in the tools menu.

            Request for Paige Kulzer - before you create a pull request, please pull the latest changes to master and rebase your IGBF-3850 branch on top (your 3850 branch is missing your changes, including version change, from IGBF-3852).

            Show
            nfreese Nowlan Freese added a comment - Tested Paige's branch on Mac. App's icon now appears in the tools menu. Request for Paige Kulzer - before you create a pull request, please pull the latest changes to master and rebase your IGBF-3850 branch on top (your 3850 branch is missing your changes, including version change, from IGBF-3852 ).
            nfreese Nowlan Freese made changes -
            Assignee Nowlan Freese [ nfreese ] Paige Kulzer [ pkulzer ]
            nfreese Nowlan Freese made changes -
            Status First Level Review in Progress [ 10301 ] To-Do [ 10305 ]
            Show
            pkulzer Paige Kulzer added a comment - Done! PR: https://bitbucket.org/lorainelab/snp-file-converter/pull-requests/15
            pkulzer Paige Kulzer made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            pkulzer Paige Kulzer made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            pkulzer Paige Kulzer made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            pkulzer Paige Kulzer made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            pkulzer Paige Kulzer made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            pkulzer Paige Kulzer made changes -
            Assignee Paige Kulzer [ pkulzer ] Ann Loraine [ aloraine ]
            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 ]
            Hide
            ann.loraine Ann Loraine added a comment -

            PR is merged and new jar file for the new version (23andMe-snp-converter-10.0.1.jar) is deployed to the Downloads section and ready for testing.

            Show
            ann.loraine Ann Loraine added a comment - PR is merged and new jar file for the new version (23andMe-snp-converter-10.0.1.jar) is deployed to the Downloads section and ready for testing.
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            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 bitbucket download page for SNP converter.

            App's icon now appears in the tools menu.

            Closing ticket.

            Show
            nfreese Nowlan Freese added a comment - Tested using bitbucket download page for SNP converter. App's icon now appears in the tools menu. Closing ticket.
            nfreese Nowlan Freese made changes -
            Assignee Paige Kulzer [ pkulzer ]
            nfreese Nowlan Freese made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

              People

              • Assignee:
                pkulzer Paige Kulzer
                Reporter:
                pkulzer Paige Kulzer
              • Votes:
                0 Vote for this issue
                Watchers:
                Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: