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

Add new parameter to NCBI Primer App

    Details

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

      Description

      NCBI primer site has added a new parameter. To ensure IGB users have access to the newly available options, we should add this new option to NCBI Primer App.

      See: https://ncbiinsights.ncbi.nlm.nih.gov/2020/04/24/primer-blast-3-match/

        Attachments

          Issue Links

            Activity

            ann.loraine Ann Loraine created issue -
            ann.loraine Ann Loraine made changes -
            Field Original Value New Value
            Epic Link IGBF-2241 [ 18388 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Summary Add new parameter Add new parameter to NCBI Primer App
            ann.loraine Ann Loraine made changes -
            Epic Link IGBF-2241 [ 18388 ] IGBF-1908 [ 17998 ]
            ann.loraine Ann Loraine made changes -
            Assignee Ann Loraine [ aloraine ]
            noor91zahara Noor Zahara (Inactive) made changes -
            Assignee Noor Zahara [ noor91zahara ]
            noor91zahara Noor Zahara (Inactive) made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            Hide
            noor91zahara Noor Zahara (Inactive) added a comment - - edited

            I have designed the GUI for the new parameter. I have attached. Let me know if it looks fine.

            I had a doubt. I do need to add the entire block that I have attached right?

            Show
            noor91zahara Noor Zahara (Inactive) added a comment - - edited I have designed the GUI for the new parameter. I have attached. Let me know if it looks fine. I had a doubt. I do need to add the entire block that I have attached right?
            noor91zahara Noor Zahara (Inactive) made changes -
            Hide
            ann.loraine Ann Loraine added a comment -

            Please also attach an image showing the entire panel.

            Show
            ann.loraine Ann Loraine added a comment - Please also attach an image showing the entire panel.
            noor91zahara Noor Zahara (Inactive) made changes -
            Hide
            noor91zahara Noor Zahara (Inactive) added a comment -

            I have attached the panel.

            Show
            noor91zahara Noor Zahara (Inactive) added a comment - I have attached the panel.
            Hide
            ann.loraine Ann Loraine added a comment -

            Can you make it more compact so that it fits the available space?

            For example, the long sentence "Primer pair....DNA" could wrap around to fit better.

            Show
            ann.loraine Ann Loraine added a comment - Can you make it more compact so that it fits the available space? For example, the long sentence "Primer pair....DNA" could wrap around to fit better.
            Hide
            nfreese Nowlan Freese added a comment -

            Possible problem with submitting accessions in place of sequence to resolve the issue with exon junction span option:

            Some of the IGB genomes (human, mouse, rat) have the NCBI accession number in the id field for each gene (NM_000133.3). But many of the other genomes (Arabidopsis) do not have the accession as the id field, since these gene annotations came from other sources (not NCBI).

            The exon junction span will work with just sequence if that sequence matches exactly with the NCBI sequence annotation for that gene. However, the tested gene in IGB Human 2013 (NM_000133.3) has a few base pair differences at the start and end of the annotation, which causes it to fail in primer blast.

            Show
            nfreese Nowlan Freese added a comment - Possible problem with submitting accessions in place of sequence to resolve the issue with exon junction span option: Some of the IGB genomes (human, mouse, rat) have the NCBI accession number in the id field for each gene (NM_000133.3). But many of the other genomes (Arabidopsis) do not have the accession as the id field, since these gene annotations came from other sources (not NCBI). The exon junction span will work with just sequence if that sequence matches exactly with the NCBI sequence annotation for that gene. However, the tested gene in IGB Human 2013 (NM_000133.3) has a few base pair differences at the start and end of the annotation, which causes it to fail in primer blast.
            Hide
            noor91zahara Noor Zahara (Inactive) added a comment -

            I was testing for different species and looks like it works fine for genome sequence for Rattus norvegicus from IGB. So I have just handled the error it throws when no primers are found with exon/intron.

            Code Changes - https://bitbucket.org/noorzahara/ncbi-primer-local/branch/IGBF-2348#diff

            Show
            noor91zahara Noor Zahara (Inactive) added a comment - I was testing for different species and looks like it works fine for genome sequence for Rattus norvegicus from IGB. So I have just handled the error it throws when no primers are found with exon/intron. Code Changes - https://bitbucket.org/noorzahara/ncbi-primer-local/branch/IGBF-2348#diff
            noor91zahara Noor Zahara (Inactive) made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            noor91zahara Noor Zahara (Inactive) made changes -
            Assignee Noor Zahara [ noor91zahara ]
            ann.loraine Ann Loraine made changes -
            Sprint Spring 8 : 24 Apr to 8 May [ 93 ] Spring 8 : 24 Apr to 8 May, Spring 8 : 11 May to 25 May [ 93, 94 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            Hide
            noor91zahara Noor Zahara (Inactive) added a comment -

            I guess the test cases written currently tests my changes as well. So there is no need for new test cases.

            Show
            noor91zahara Noor Zahara (Inactive) added a comment - I guess the test cases written currently tests my changes as well. So there is no need for new test cases.
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Assignee Pooja Nikhare [ pooja.nikhare ]
            Hide
            pooja.nikhare Pooja Nikhare (Inactive) added a comment - - edited

            Observations:
            Following features are working fine:

            • Exon/ Intron options are set to default options (as in NCBI site) and properly sent while sending a request to NCBI set.
            • Options are set default by "Reset Default" Option.

            Observed the following issues:

            • When Exon Junction span is set to default "No Preference". Primer blast for all seq and organism works fine.
              But when it is set to other 2 options - "Primer must span an exon-exon junction", "Primer may not span an exon-exon junction" for few of the sym exception is being thrown. As for these options, NCBI Site redirects to an Info page , the Info page details are to be handled.
              Can be reproduced by testing with Rat genome - Taar7a .
            • According to the NCBI site A refseq mRNA seq is required for exon/intron selection. Can we send default options of exon/intron option as "No Preference" if the database type selected is other than "refseq mRNA" ?
              Because other combination ends up in info page, which user has to see in console.
            • Few parameters have been added in NCBI site like Max target size, Allow splice variants under Primer Pair Specificity Checking Parameters. And Post processing option is removed.

            If required test cases for different options of Exon Junction span can be added. Test case exists for default options "No preference".

            Show
            pooja.nikhare Pooja Nikhare (Inactive) added a comment - - edited Observations: Following features are working fine: Exon/ Intron options are set to default options (as in NCBI site) and properly sent while sending a request to NCBI set. Options are set default by "Reset Default" Option. Observed the following issues: When Exon Junction span is set to default "No Preference". Primer blast for all seq and organism works fine. But when it is set to other 2 options - "Primer must span an exon-exon junction", "Primer may not span an exon-exon junction" for few of the sym exception is being thrown. As for these options, NCBI Site redirects to an Info page , the Info page details are to be handled. Can be reproduced by testing with Rat genome - Taar7a . According to the NCBI site A refseq mRNA seq is required for exon/intron selection. Can we send default options of exon/intron option as "No Preference" if the database type selected is other than "refseq mRNA" ? Because other combination ends up in info page, which user has to see in console. Few parameters have been added in NCBI site like Max target size, Allow splice variants under Primer Pair Specificity Checking Parameters. And Post processing option is removed. If required test cases for different options of Exon Junction span can be added. Test case exists for default options "No preference".
            Show
            noor91zahara Noor Zahara (Inactive) added a comment - For this ticket. kindly use branch IGBF-2320 . https://bitbucket.org/noorzahara/ncbi-primer-local/branch/IGBF-2320#diff
            noor91zahara Noor Zahara (Inactive) made changes -
            Link This issue blocks IGBF-2320 [ IGBF-2320 ]
            Hide
            pooja.nikhare Pooja Nikhare (Inactive) added a comment -

            Tested branch: https://bitbucket.org/noorzahara/ncbi-primer-local/branch/IGBF-2320

            • Cloned the repo. Ran "mvn install", builds the NCBI app properly without exception.
            • Added https://bitbucket.org/noorzahara/ncbi-primer-local/downloads/ as a repository.
              Install the app. NCBI App got installed and the Primer option tab can be seen in bottom tab.
            • Select a genome. To test default options of Exon junction span options ("No preference")
              select syms, right click on primer blast option. It opens the NCBI primer blast site.
              -Tested different param values for exon/intron selection panel, database, and other genomes. If the parameters are valid NCBI Primer site opens else Error or Info page is captured in console without exception.
            • Click on "Reset Default". All options are set to default.
            Show
            pooja.nikhare Pooja Nikhare (Inactive) added a comment - Tested branch: https://bitbucket.org/noorzahara/ncbi-primer-local/branch/IGBF-2320 Cloned the repo. Ran "mvn install", builds the NCBI app properly without exception. Added https://bitbucket.org/noorzahara/ncbi-primer-local/downloads/ as a repository. Install the app. NCBI App got installed and the Primer option tab can be seen in bottom tab. Select a genome. To test default options of Exon junction span options ("No preference") select syms, right click on primer blast option. It opens the NCBI primer blast site. -Tested different param values for exon/intron selection panel, database, and other genomes. If the parameters are valid NCBI Primer site opens else Error or Info page is captured in console without exception. Click on "Reset Default". All options are set to default.
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Assignee Pooja Nikhare [ pooja.nikhare ] Noor Zahara [ noor91zahara ]
            Show
            noor91zahara Noor Zahara (Inactive) added a comment - PR submitted https://bitbucket.org/lorainelab/ncbi-primer/pull-requests/1/igbf-2320/diff
            noor91zahara Noor Zahara (Inactive) made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            noor91zahara Noor Zahara (Inactive) made changes -
            Assignee Noor Zahara [ noor91zahara ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
            Hide
            pooja.nikhare Pooja Nikhare (Inactive) added a comment -
            Show
            pooja.nikhare Pooja Nikhare (Inactive) added a comment - Added https://bitbucket.org/lorainelab/ncbi-primer/downloads/ (team repos downloads) as a repository in IGB. New parameters can be seen in Exon/intron selection panel under primer options tab. Tested for different parameter values. Moving the ticket to Done.
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

              People

              • Assignee:
                Unassigned
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: