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

Fix Color By Property matching any part of numeric value

        Details

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

          Description

          Situation: While adding additional properties to the Color By feature in IGB (IGBF-3640) we found that the Color By > Property behaved oddly. For example, if no value is entered for the property > id then everything is counted as being a match. Or, for example, if an id contains the string AT1G29990 and the value searched for is AT1G2, then all annotations with AT1G2 would be identified.

          Task: Determine if this is the intended behavior. While this may make sense for searching for strings, it may be confusing for users with the expanded properties now including numeric values.

          To reproduce the issue:

          1. Start IGB
          2. Open the A_thaliana_Jun_2009 genome
          3. Right-click on the Araport track name
          4. Select Color By...
          5. In the Color By dropdown, select Property
          6. In the property dropdown, select id
          7. Enter a value of AT1G2
          8. Click OK

            Attachments

              Issue Links

              relates to

              Improvement - An improvement or enhancement to an existing feature or task. IGBF-3640 Expand color by and Filter by to include the props

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                jsirigin Jaya Sravani Sirigineedi added a comment - - edited

                Investigated the issue and fixed it for numeric values. Updated code is available at branch: https://bitbucket.org/jaya-sravani/integrated-genome-browser/branch/IGBF-3722. Please review and let me know if there any issues.



                To Test:

                1. Build the jar file for the branch in your local or download the installer.
                2. Start the application, select any species, and select any one of the ucsc rest tracks that has extra properties (bed, bigBed, bedDetail, and bigGenePRed track types have extra properties).
                3. Load data for the track and right-click on the track name to see the options menu
                4. Click on Color by option and select property from the dropdown items
                5. Now, in the property dropdown list, you should be able to see the extra properties that are present in that track
                6. Select any property whose value is numeric
                7. Test the value by giving the exact number that any of the displayed annotations have, and giving part of the number and giving a number that none of the annotations have.
                Show
                jsirigin Jaya Sravani Sirigineedi added a comment - - edited Investigated the issue and fixed it for numeric values. Updated code is available at branch: https://bitbucket.org/jaya-sravani/integrated-genome-browser/branch/IGBF-3722 . Please review and let me know if there any issues. To Test: Build the jar file for the branch in your local or download the installer. Start the application, select any species, and select any one of the ucsc rest tracks that has extra properties (bed, bigBed, bedDetail, and bigGenePRed track types have extra properties). Load data for the track and right-click on the track name to see the options menu Click on Color by option and select property from the dropdown items Now, in the property dropdown list, you should be able to see the extra properties that are present in that track Select any property whose value is numeric Test the value by giving the exact number that any of the displayed annotations have, and giving part of the number and giving a number that none of the annotations have.
                Hide
                Permalink
                nfreese Nowlan Freese added a comment -

                Tested on Mac using Sravani's branch.

                Color by... Property for numeric values now matches only exact values. Strings still match when only a partial match is found (after discussing with Sravani, we decided that this behavior is convenient for users).

                Ready for pull request.

                Show
                nfreese Nowlan Freese added a comment - Tested on Mac using Sravani's branch. Color by... Property for numeric values now matches only exact values. Strings still match when only a partial match is found (after discussing with Sravani, we decided that this behavior is convenient for users). Ready for pull request.
                Hide
                Permalink
                jsirigin Jaya Sravani Sirigineedi added a comment - - edited

                Raised the pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1011. Please review and let me know if there are any issues.

                Show
                jsirigin Jaya Sravani Sirigineedi added a comment - - edited Raised the pull request: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/1011 . Please review and let me know if there are any issues.
                Hide
                Permalink
                ann.loraine Ann Loraine added a comment - - edited

                PR is merged. Installers are build and deployed to bioviz.org "early access" section for testing.

                Show
                ann.loraine Ann Loraine added a comment - - edited PR is merged. Installers are build and deployed to bioviz.org "early access" section for testing.
                Hide
                Permalink
                nfreese Nowlan Freese added a comment -

                Tested main branch installer on Mac.

                Color by... Property for numeric values now matches only exact values. Strings still match when only a partial match is found.

                Closing ticket.

                Show
                nfreese Nowlan Freese added a comment - Tested main branch installer on Mac. Color by... Property for numeric values now matches only exact values. Strings still match when only a partial match is found. Closing ticket.

                  People

                  • Assignee:
                    jsirigin Jaya Sravani Sirigineedi
                    Reporter:
                    nfreese Nowlan Freese
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    3 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: