Details

    • Type: Epic
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Epic Name:
      Refactor NCBI Primer App

      Description

      Several years ago we wrote an App called "NCBI Primer design" that lets users run a primer design search using a service hosted at NCBI - National Center for Biotechnology Information.

      The source code is located in this repository: https://bitbucket.org/lorainelab/experimental-apps-for-igb

      Recently, we updated the code to use SCR instead of an Activator class to plug into IGB. Now, if you want to use the App, you just install it in the usual way, right-click a gene model, and choose the "NCBI" option from the menu. What happens next is that the App sends a POST query to NCBI with a bunch of parameters. Users within IGB can modify the parameters by selecting File > Preferences > Primer Options. However, the parameters accepted by NCBI have changed, and we need to update them in the Primer Options GUI.

      Please note: Ann Loraine added a number of comments and logging statements to the code.

      Another issue is that we do not yet have good unit test coverage for the App.

      One problem with this is that we have a class/interface called NCBIPrimerBlastOpts that has a single method getOptions that returns a RemotePrimerBlastNCBI.PrimerBlastOptions object. I think it would be better to move PrimerBlastOptions out of the class RemotePrimerBlastNCBI and create an implementation that captures reasonable defaults. Then, the GUI could obtain a reference to such an object and use it to populate default fields for the GUI. We could then use this object in the testing code. Currently, there is no way to do that – at least not as I can see!

      Another issue: There is some older code left over from the Apollo project (now defunct) in which the output of the NCBI tool was parsed and added to the Apollo GUI. Let's get rid of all this old code.

      Another issue: the Activator class needs to be removed now that we are using SCR.

      The GUI should be moved out of the Preferences window into a tabbed panel. I think this is a more natural place for users to find the App. Suggestion: Use the Netbeans GUI builder to design the new GUI. It will likely be easier than coding by hand and will likely produce a more attractive result.

      This is a pretty advanced task because it involves a major re-write and re-factoring of this code. It might make good sense to create an entirely new project in a separate repository for this update.

        Attachments

          Issue Links

            Activity

            ann.loraine Ann Loraine created issue -
            ann.loraine Ann Loraine made changes -
            Field Original Value New Value
            Epic Link IGBF-1908 [ 17998 ]
            ann.loraine Ann Loraine made changes -
            Link This issue relates to IGBF-2213 [ IGBF-2213 ]
            ann.loraine Ann Loraine made changes -
            Rank Ranked higher
            ann.loraine Ann Loraine made changes -
            Rank Ranked lower
            prutha Prutha Kulkarni (Inactive) made changes -
            Assignee Prutha Kulkarni [ prutha ]
            Hide
            ann.loraine Ann Loraine added a comment -

            I think we should move all the classes into one package - creating ticket for same and assigning to Pooja.

            Show
            ann.loraine Ann Loraine added a comment - I think we should move all the classes into one package - creating ticket for same and assigning to Pooja.
            Hide
            ann.loraine Ann Loraine added a comment -

            Converting this to an Epic.

            Show
            ann.loraine Ann Loraine added a comment - Converting this to an Epic.
            ann.loraine Ann Loraine made changes -
            Issue Type Task [ 3 ] Epic [ 10000 ]
            Epic Name Refactor NCBI Primer App
            Epic Status To Do [ 10001 ]
            ann.loraine Ann Loraine made changes -
            Epic Link IGBF-1908 [ 17998 ]
            ann.loraine Ann Loraine made changes -
            Assignee Prutha Kulkarni [ prutha ]
            ann.loraine Ann Loraine made changes -
            Sprint Spring 3 : 3 Feb to 14 Feb [ 86 ]
            ann.loraine Ann Loraine made changes -
            Story Points 3
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2255 [ 18402 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2256 [ 18403 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2257 [ 18404 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Epic Child IGBF-2258 [ 18405 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Epic Child IGBF-2259 [ 18406 ]
            prutha Prutha Kulkarni (Inactive) made changes -
            Epic Child IGBF-2261 [ 18408 ]
            pooja.nikhare Pooja Nikhare (Inactive) made changes -
            Epic Child IGBF-2287 [ 18441 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2317 [ 18471 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2319 [ 18473 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2320 [ 18474 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2328 [ 18482 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2348 [ 18504 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2348 [ 18504 ]
            ann.loraine Ann Loraine made changes -
            Epic Child IGBF-2319 [ 18473 ]
            ann.loraine Ann Loraine made changes -
            Epic Status To Do [ 10001 ] Done [ 10003 ]
            ann.loraine Ann Loraine made changes -
            Status To-Do [ 10305 ] In Progress [ 3 ]
            ann.loraine Ann Loraine made changes -
            Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
            ann.loraine Ann Loraine made changes -
            Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
            ann.loraine Ann Loraine made changes -
            Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
            ann.loraine Ann Loraine made changes -
            Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
            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 ]
            ann.loraine Ann Loraine made changes -
            Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
            ann.loraine Ann Loraine 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:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: