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

Stop zoom stripe label from blocking residues in Coordinates Track

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
    • Story Points:
      3
    • Sprint:
      Winter 2018 Sprint 3, Spring 2019 Sprint 1, Spring 2019 Sprint 2, Spring 2019 Sprint 3, Spring 2019 Sprint 4, Spring 2019 Sprint 5

      Description

      The label for the zoom stripe covers up the residues nearest the zoom stripe.

      Users often have to toggle the zoom stripe label on and off if they want to know both the residue and the exact location.

      If the zoom stripe label appeared slightly higher in the window, users would be able to see the coordinate and the residue at the same time.

      Consult with Dr. Freese for detailed description of the problem & potential solution.

      Task is:

      • Understand & better document (as required) the problem
      • Investigate potential solutions
      • Make note of classes and packages involved

      Note: Labeled Intermediate level difficulty, but this is a provisional assessment. (May need to re-label as Advanced depending on results of above.)

        Attachments

          Issue Links

            Activity

            Hide
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - - edited

            Issue :- Zoom stripe label blocking the residue sequence.

            Solution :- Moved the zoom stripe label up on the Y-axis to prevent it from overlapping the genome residue sequence.

            Files changed:-
            genoviz_sdk/pom.xml
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/StringGlyph.java
            genoviz_sdk/src/main/java/com/affymetrix/genoviz/widget/Shadow.java
            pom.xml

            Testing steps:-
            1) Click on any genome to load the sequence in IGB project.
            2) Click on Load Sequence button from top right corner to load the genome residue.
            3) Click anywhere to see the zoom stripe label.

            Test results:-
            Please find the attached screenshots for test results.

            Code changes:-
            https://bitbucket.org/rpatil14/riddhi-genovizsdk/branch/IGBF-1230#diff

            Note:- Changed the genoviz_sdk and genovizTutorial jar verrsion from 1.7 to 1.8. We need to change pom.xml of IGB project to use 1.8 version og genoviz module. So, the file pom.xml of IGB project will be committed once, these changes are pushed to the team repository in order to prevent any breakage.

            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - - edited Issue :- Zoom stripe label blocking the residue sequence. Solution :- Moved the zoom stripe label up on the Y-axis to prevent it from overlapping the genome residue sequence. Files changed:- genoviz_sdk/pom.xml genoviz_sdk/src/main/java/com/affymetrix/genoviz/glyph/StringGlyph.java genoviz_sdk/src/main/java/com/affymetrix/genoviz/widget/Shadow.java pom.xml Testing steps:- 1) Click on any genome to load the sequence in IGB project. 2) Click on Load Sequence button from top right corner to load the genome residue. 3) Click anywhere to see the zoom stripe label. Test results:- Please find the attached screenshots for test results. Code changes:- https://bitbucket.org/rpatil14/riddhi-genovizsdk/branch/IGBF-1230#diff Note:- Changed the genoviz_sdk and genovizTutorial jar verrsion from 1.7 to 1.8. We need to change pom.xml of IGB project to use 1.8 version og genoviz module. So, the file pom.xml of IGB project will be committed once, these changes are pushed to the team repository in order to prevent any breakage.
            Hide
            ann.loraine Ann Loraine added a comment -

            For the next step:

            • Rebase your topic branch on the latest Genoviz SDK maste branch
            • Issue pull request from your Genoviz SDK fork/branch to team Genoviz SDK repository master branch, move this issue to "Pull Request Submitted"
            • Make a branch on your IGB fork for issue IGBF-1230
            • In that branch only, update the POM.xml to use Genoviz SDK 1.8
            • Push your changes.

            Once you push your changes and I have deployed Genoviz SDK 1.8 onto Loraine Lab Nexus (https://nexus.bioviz.org), the installer should get built on your fork, and we can test how it looks. However, the build on your fork should fail before I deploy Genoviz SDK 1.8. Build failure in this case would be a good test of our dependency management system. So keep an eye on your build process to make sure system is behaving as expected.

            Show
            ann.loraine Ann Loraine added a comment - For the next step: Rebase your topic branch on the latest Genoviz SDK maste branch Issue pull request from your Genoviz SDK fork/branch to team Genoviz SDK repository master branch, move this issue to "Pull Request Submitted" Make a branch on your IGB fork for issue IGBF-1230 In that branch only, update the POM.xml to use Genoviz SDK 1.8 Push your changes. Once you push your changes and I have deployed Genoviz SDK 1.8 onto Loraine Lab Nexus ( https://nexus.bioviz.org ), the installer should get built on your fork, and we can test how it looks. However, the build on your fork should fail before I deploy Genoviz SDK 1.8. Build failure in this case would be a good test of our dependency management system. So keep an eye on your build process to make sure system is behaving as expected.
            Hide
            ann.loraine Ann Loraine added a comment -

            Testing:

            • Examine how new zoom stripe looks in all places where it is being used (that we know about).

            This includes:

            • ProtAnnot (IGB App accessible via App Manager GUI)
            • Sliced View tab in IGB
            • Main view in IGB

            I checked ProtAnnot's POM.xml
            This POM.xml does not mention the Genoviz SDK version but instead inherits that from its parent POM, which is the POM for the entire project. This means that if we update the parent POM, ProtAnnot bundle deployed on https://bioviz.org/igbserver should work fine. It is using the provided Genoviz SDK bundle in IGB.

            Show
            ann.loraine Ann Loraine added a comment - Testing: Examine how new zoom stripe looks in all places where it is being used (that we know about). This includes: ProtAnnot (IGB App accessible via App Manager GUI) Sliced View tab in IGB Main view in IGB I checked ProtAnnot's POM.xml This POM.xml does not mention the Genoviz SDK version but instead inherits that from its parent POM, which is the POM for the entire project. This means that if we update the parent POM, ProtAnnot bundle deployed on https://bioviz.org/igbserver should work fine. It is using the provided Genoviz SDK bundle in IGB.
            Hide
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment -
            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Please check the attached Test_Document_IGBF_1230.xls for the testing results and the next steps. Created pull request :- https://bitbucket.org/lorainelab/genoviz-sdk/pull-requests/11/igbf-1230-zoom-stripe-label-blocking-the/diff Updated pom.xml of IGB project to use latest version of genoviz jar https://bitbucket.org/rpatil14/riddhifork-igb/branch/IGBF-1230#diff
            Hide
            ann.loraine Ann Loraine added a comment -

            I am not able to see the images in my Excel (could be a Mac vs Windows issue)
            Can you attach the PNGs separately?

            Show
            ann.loraine Ann Loraine added a comment - I am not able to see the images in my Excel (could be a Mac vs Windows issue) Can you attach the PNGs separately?
            Hide
            ann.loraine Ann Loraine added a comment - - edited

            Merged pull request https://bitbucket.org/lorainelab/genoviz-sdk/pull-requests/11/igbf-1230-zoom-stripe-label-blocking-the/diff into master on team repository. Build failed, but was easy to fix by changing reference to parent POM version from 1.1.7 to 1.1.8 in tutorial module. Now it builds. Version 1.1.8 of library and tutorials are now deployed on https://nexus.bioviz.org.

            Ready now to increment version of of Genoviz SDK used in IGB to 1.1.8 to pick up the change in zoom stripe label location.

            Show
            ann.loraine Ann Loraine added a comment - - edited Merged pull request https://bitbucket.org/lorainelab/genoviz-sdk/pull-requests/11/igbf-1230-zoom-stripe-label-blocking-the/diff into master on team repository. Build failed, but was easy to fix by changing reference to parent POM version from 1.1.7 to 1.1.8 in tutorial module. Now it builds. Version 1.1.8 of library and tutorials are now deployed on https://nexus.bioviz.org . Ready now to increment version of of Genoviz SDK used in IGB to 1.1.8 to pick up the change in zoom stripe label location.
            Hide
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment -

            Pull request submitted for IGB project:- (Use updated genoviz jar in IGB)
            https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/663/igbf-1230-point-igb-to-use-18-version-of/diff

            Note:- I have attached the images from Test_Document_IGBF_1230.xls separately to the ticket.

            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Pull request submitted for IGB project:- (Use updated genoviz jar in IGB) https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/663/igbf-1230-point-igb-to-use-18-version-of/diff Note:- I have attached the images from Test_Document_IGBF_1230.xls separately to the ticket.
            Hide
            ann.loraine Ann Loraine added a comment -

            Please re-run pipeline for IGBF-1230.
            It should now run properly and build the installer as I have deployed the new 1.1.8 Genoviz artifacts on https://nexus.bioviz.org.

            Show
            ann.loraine Ann Loraine added a comment - Please re-run pipeline for IGBF-1230 . It should now run properly and build the installer as I have deployed the new 1.1.8 Genoviz artifacts on https://nexus.bioviz.org .
            Hide
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment -

            Build was successful for IGBF-1230

            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Build was successful for IGBF-1230
            Hide
            ann.loraine Ann Loraine added a comment -

            Ready for pull request.

            Show
            ann.loraine Ann Loraine added a comment - Ready for pull request.
            Hide
            Jill Jill Jenkins (Inactive) added a comment - - edited

            MacOS - When multiple tracks are selected, one with merged strands and one without, upon zooming - the zoom stripe label still blocks residues. Screen shot added.

            Show
            Jill Jill Jenkins (Inactive) added a comment - - edited MacOS - When multiple tracks are selected, one with merged strands and one without, upon zooming - the zoom stripe label still blocks residues. Screen shot added.
            Hide
            ann.loraine Ann Loraine added a comment -

            RP and AL discussed next steps. We realized that the best way to fix this would be to modify Shadow.java (in Genoviz SDK library) to enable users of this class to change y-position of the label (a StringGlyph) based on what it is covering in the container NeoMap. In light of this, we have decided to revert the changes to the library and re-visit this issue at a later date. AL will take care of reverting the changes and also updating version numbers as required. When that is done, we will make a new issue to modify Shadow.java (as noted) and close this.

            Show
            ann.loraine Ann Loraine added a comment - RP and AL discussed next steps. We realized that the best way to fix this would be to modify Shadow.java (in Genoviz SDK library) to enable users of this class to change y-position of the label (a StringGlyph) based on what it is covering in the container NeoMap. In light of this, we have decided to revert the changes to the library and re-visit this issue at a later date. AL will take care of reverting the changes and also updating version numbers as required. When that is done, we will make a new issue to modify Shadow.java (as noted) and close this.
            Hide
            ann.loraine Ann Loraine added a comment -

            After using the new feature in 9.1.0, it appears to be working reasonably well and is a big improvement in usability. In light of this, this issue is now closed.

            Show
            ann.loraine Ann Loraine added a comment - After using the new feature in 9.1.0, it appears to be working reasonably well and is a big improvement in usability. In light of this, this issue is now closed.

              People

              • Assignee:
                ann.loraine Ann Loraine
                Reporter:
                ieclabau Ivory Blakley (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: