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

Improve Make scaled coverage graph interface

    Details

    • Type: Task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None
    • Story Points:
      3
    • Sprint:
      Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10, Fall 3 2021 Sep 13 - Sep 24, Fall 4 2021 Sep 27 - Oct 8, Fall 5 2021 Oct 11 - Oct 22

      Description

      The "make scaled coverage graph" interface has several question mark (help) icons next to text entry boxes.

      When I tried to click them to view help information, nothing happened. Hovering the mouse over them also did nothing. They do not seem to have any function.

      If these question marks do nothing, they should be removed. Were these meant as a placeholders? If yes, that's fine, but leaving them without any function is not good - it undermines users' trust that the site is reliable and well-designed.

        Attachments

          Activity

          ann.loraine Ann Loraine created issue -
          ann.loraine Ann Loraine made changes -
          Field Original Value New Value
          Epic Link IGBF-2376 [ 18533 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Sorry, I was wrong about their not doing anything. I just needed to hover the mouse over them a bit longer. When I did that, a tooltip appeared.

          However, most of the tooltips are not very useful. They simply repeat the label, using slightly different wording.

          The only tooltip that introduces new information is the tooltip for "Output file name". This tooltip tells the user about where the output file will be saved.

          However, there is a problem with putting this information here. The tooltip pertains to the "Output file name" textbox, not the location of the file, which is not something the user can change, so far as I know.

          I think this crucial information about where the results files and folders will be saved needs to be added to the interface, not hidden in a tooltip.

          Therefore I think we should display this information in another part of the interface. I think it should be added to a new section below the "Run Analysis" button.

          Other requests:

          • The question mark icons are too close to the preceding letters. Can we add some some whitespace so that the interface looks less crowded?
          • Let's also improve the tooltips, making them easier to read and more substantial looking. Can we make them look like the new tooltip for the heading on the "Manage Link" right panel screen?
          Show
          ann.loraine Ann Loraine added a comment - - edited Sorry, I was wrong about their not doing anything. I just needed to hover the mouse over them a bit longer. When I did that, a tooltip appeared. However, most of the tooltips are not very useful. They simply repeat the label, using slightly different wording. The only tooltip that introduces new information is the tooltip for "Output file name". This tooltip tells the user about where the output file will be saved. However, there is a problem with putting this information here. The tooltip pertains to the "Output file name" textbox, not the location of the file, which is not something the user can change, so far as I know. I think this crucial information about where the results files and folders will be saved needs to be added to the interface, not hidden in a tooltip. Therefore I think we should display this information in another part of the interface. I think it should be added to a new section below the "Run Analysis" button. Other requests: The question mark icons are too close to the preceding letters. Can we add some some whitespace so that the interface looks less crowded? Let's also improve the tooltips, making them easier to read and more substantial looking. Can we make them look like the new tooltip for the heading on the "Manage Link" right panel screen?
          ann.loraine Ann Loraine made changes -
          Summary Make the help (question mark) icons do something Improve scaled Make scaled coverage graph interface
          ann.loraine Ann Loraine made changes -
          Assignee Karthik Raveendran [ karthik ]
          karthik Karthik Raveendran made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          karthik Karthik Raveendran made changes -
          Assignee Karthik Raveendran [ karthik ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1 2021 Aug 16 - Aug 27 [ 127 ] Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Aug 10 [ 127, 128 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          karthik Karthik Raveendran made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          karthik Karthik Raveendran made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          karthik Karthik Raveendran added a comment - - edited

          Dr. [~aloraine], Ill need some help updating the database in the production server for the tooltips (maybe with the Ansible playbook). I need to run the following queries in the inputdetails table:

          update inputdetails
          set htmldata = replace(htmldata,"title='Output folder saved to current folder if in Home, saved to Analyses folder if data in Shared/Community'",
          "title='Output file will be saved in current folder if you are in Home. If you are in Shared or Community, it will be saved in the analyses folder in Home'");

          update inputdetails
          set htmldata = replace(htmldata,"title='File output to current directory if in Home or Analyses folder if data from Shared or Community'",
          "title='Output file will be saved in current folder if you are in Home. If you are in Shared or Community, it will be saved in the analyses folder in Home'");

          update inputdetails
          set htmldata = replace(htmldata,"title='File is output to current directory if in Home or output to Analyses if file in Community/Shared'",
          "title='Output file will be saved in current folder if you are in Home. If you are in Shared or Community, it will be saved in the analyses folder in Home'");

          Show
          karthik Karthik Raveendran added a comment - - edited Dr. [~aloraine] , Ill need some help updating the database in the production server for the tooltips (maybe with the Ansible playbook). I need to run the following queries in the inputdetails table: update inputdetails set htmldata = replace(htmldata,"title='Output folder saved to current folder if in Home, saved to Analyses folder if data in Shared/Community'", "title='Output file will be saved in current folder if you are in Home. If you are in Shared or Community, it will be saved in the analyses folder in Home'"); update inputdetails set htmldata = replace(htmldata,"title='File output to current directory if in Home or Analyses folder if data from Shared or Community'", "title='Output file will be saved in current folder if you are in Home. If you are in Shared or Community, it will be saved in the analyses folder in Home'"); update inputdetails set htmldata = replace(htmldata,"title='File is output to current directory if in Home or output to Analyses if file in Community/Shared'", "title='Output file will be saved in current folder if you are in Home. If you are in Shared or Community, it will be saved in the analyses folder in Home'");
          Hide
          ann.loraine Ann Loraine added a comment -

          Replying to: Karthik Raveendran
          The above queries can be done, but I'm concerned that query success depends on knowing the current value for a row. Instead, it would be better to query based on the App name, or some other value that can function as a primary key.
          Can you give me a little more details regarding your strategy for making this solution work on every test server, regardless of the current values in the database?

          Show
          ann.loraine Ann Loraine added a comment - Replying to: Karthik Raveendran The above queries can be done, but I'm concerned that query success depends on knowing the current value for a row. Instead, it would be better to query based on the App name, or some other value that can function as a primary key. Can you give me a little more details regarding your strategy for making this solution work on every test server, regardless of the current values in the database?
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10 [ 127, 128 ] Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10, Fall 3 2021Sep 13 - Sep 24 [ 127, 128, 129 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          nfreese Nowlan Freese made changes -
          Summary Improve scaled Make scaled coverage graph interface Improve Make scaled coverage graph interface
          karthik Karthik Raveendran made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          Show
          nfreese Nowlan Freese added a comment - Merged https://bitbucket.org/nfreese/bioviz-connect/pull-requests/255/igbf-2923-refactor-analyses-ui-and?link_source=email
          nfreese Nowlan Freese made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          nfreese Nowlan Freese made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          nfreese Nowlan Freese made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          nfreese Nowlan Freese made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          nfreese Nowlan Freese made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          nfreese Nowlan Freese made changes -
          Assignee Karthik Raveendran [ karthik ]
          Hide
          karthik Karthik Raveendran added a comment -

          Submitted PR. Commit

          Testing: [~aloraine], it would be great if you could deploy this in the test site. My recommendation is that this commit should be rigorously tested before push to production because the analysis module is quite difficult to test while development as the results of the analysis comes back after a significant delay and this commit has a lot of files with a lot of changes.

          Show
          karthik Karthik Raveendran added a comment - Submitted PR. Commit Testing: [~aloraine] , it would be great if you could deploy this in the test site. My recommendation is that this commit should be rigorously tested before push to production because the analysis module is quite difficult to test while development as the results of the analysis comes back after a significant delay and this commit has a lot of files with a lot of changes.
          Hide
          ann.loraine Ann Loraine added a comment -

          Latest main branch from Nowlan Freese repository deployed to bvctest3.bioviz.org.
          attn: Karthik Raveendran and Nowlan Freese

          Show
          ann.loraine Ann Loraine added a comment - Latest main branch from Nowlan Freese repository deployed to bvctest3.bioviz.org. attn: Karthik Raveendran and Nowlan Freese
          nfreese Nowlan Freese made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ]
          Hide
          nfreese Nowlan Freese added a comment -

          Testing:

          For the "Make scaled coverage graph", the Input File Index (bai) is no longer being autofilled on bvctest3.

          The analysis jobs are failing at "error downloading input_path.list". When I looked at the input param_value the path was listed as "/iplant/home/nowlanf/testFiles/!howLong?//48v37_summits.fasta". Note the double slashes.

          Show
          nfreese Nowlan Freese added a comment - Testing: For the "Make scaled coverage graph", the Input File Index (bai) is no longer being autofilled on bvctest3. The analysis jobs are failing at "error downloading input_path.list". When I looked at the input param_value the path was listed as "/iplant/home/nowlanf/testFiles/!howLong?//48v37_summits.fasta". Note the double slashes.
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ] Karthik Raveendran [ karthik ]
          nfreese Nowlan Freese made changes -
          Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10, Fall 3 2021 Sep 13 - Sep 24 [ 127, 128, 129 ] Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10, Fall 3 2021 Sep 13 - Sep 24, Fall 4 2021 Sep 27 - Oct 8 [ 127, 128, 129, 130 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          karthik Karthik Raveendran made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          karthik Karthik Raveendran made changes -
          Status In Progress [ 3 ] To-Do [ 10305 ]
          karthik Karthik Raveendran made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          karthik Karthik Raveendran made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          karthik Karthik Raveendran made changes -
          Assignee Karthik Raveendran [ karthik ]
          Hide
          karthik Karthik Raveendran added a comment -

          Submitted PR for first level review
          Commit Includes: File path correction and auto-filling index file

          Show
          karthik Karthik Raveendran added a comment - Submitted PR for first level review Commit Includes: File path correction and auto-filling index file
          Show
          nfreese Nowlan Freese added a comment - Merged https://bitbucket.org/nfreese/bioviz-connect/pull-requests/256/igbf-2923-fix-filepath-error-and-add-index?link_source=email
          nfreese Nowlan Freese made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          nfreese Nowlan Freese made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          nfreese Nowlan Freese made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          nfreese Nowlan Freese made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          nfreese Nowlan Freese made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          ann.loraine Ann Loraine made changes -
          Sprint Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10, Fall 3 2021 Sep 13 - Sep 24, Fall 4 2021 Sep 27 - Oct 8 [ 127, 128, 129, 130 ] Fall 1 2021 Aug 16 - Aug 27, Fall 2 2021 Aug 30 - Sep10, Fall 3 2021 Sep 13 - Sep 24, Fall 4 2021 Sep 27 - Oct 8, Fall 5 2021 Oct 11 - Oct 22 [ 127, 128, 129, 130, 131 ]
          ann.loraine Ann Loraine made changes -
          Rank Ranked higher
          omarne Omkar Marne (Inactive) made changes -
          Assignee Omkar Marne [ omarne ]
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Test new functionality using testing host: https://bvctest6.bioviz.org
          For background, testers can read latest draft of the bioviz-connect paper: https://www.biorxiv.org/content/10.1101/2020.05.15.098533v4

          Show
          ann.loraine Ann Loraine added a comment - - edited Test new functionality using testing host: https://bvctest6.bioviz.org For background, testers can read latest draft of the bioviz-connect paper: https://www.biorxiv.org/content/10.1101/2020.05.15.098533v4
          nfreese Nowlan Freese made changes -
          Assignee Omkar Marne [ omarne ] Nowlan Freese [ nfreese ]
          nfreese Nowlan Freese made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          Hide
          nfreese Nowlan Freese added a comment - - edited

          Analysis jobs are now being completed successfully.
          The Input File Index (bai) is now being correctly autofilled.

          Problem:
          The "Find enriched sequence motifs" analysis is missing the input field for Output Folder Name.

          Show
          nfreese Nowlan Freese added a comment - - edited Analysis jobs are now being completed successfully. The Input File Index (bai) is now being correctly autofilled. Problem: The "Find enriched sequence motifs" analysis is missing the input field for Output Folder Name.
          nfreese Nowlan Freese made changes -
          Status Post-merge Testing In Progress [ 10003 ] To-Do [ 10305 ]
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ] Karthik Raveendran [ karthik ]
          karthik Karthik Raveendran made changes -
          Status To-Do [ 10305 ] In Progress [ 3 ]
          Hide
          karthik Karthik Raveendran added a comment -

          Pushed changes to repo. Commit in branch IGBF-2923

          Th conditional statement that handles input fields did not include FolderOutput field which is added in this commit.

          Test Protocol:
          Search for a .fasta file in Home directory and run Analyse from right-click menu. Click on the "Find enriched sequence motifs" button and check if the Output Folder Name field is present. Try running the analysis and if it appears as 'Submitted' in the Analysis History page

          Show
          karthik Karthik Raveendran added a comment - Pushed changes to repo. Commit in branch IGBF-2923 Th conditional statement that handles input fields did not include FolderOutput field which is added in this commit. Test Protocol: Search for a .fasta file in Home directory and run Analyse from right-click menu. Click on the "Find enriched sequence motifs" button and check if the Output Folder Name field is present. Try running the analysis and if it appears as 'Submitted' in the Analysis History page
          karthik Karthik Raveendran made changes -
          Assignee Karthik Raveendran [ karthik ]
          karthik Karthik Raveendran made changes -
          Status In Progress [ 3 ] Needs 1st Level Review [ 10005 ]
          nfreese Nowlan Freese made changes -
          Status Needs 1st Level Review [ 10005 ] First Level Review in Progress [ 10301 ]
          nfreese Nowlan Freese made changes -
          Status First Level Review in Progress [ 10301 ] Ready for Pull Request [ 10304 ]
          karthik Karthik Raveendran made changes -
          Status Ready for Pull Request [ 10304 ] Pull Request Submitted [ 10101 ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Passes review as per discussion during Oct 19 scrum meeting.

          Show
          ann.loraine Ann Loraine added a comment - Passes review as per discussion during Oct 19 scrum meeting.
          nfreese Nowlan Freese made changes -
          Status Pull Request Submitted [ 10101 ] Reviewing Pull Request [ 10303 ]
          Show
          nfreese Nowlan Freese added a comment - Merged https://bitbucket.org/nfreese/bioviz-connect/pull-requests/261/igbf-2923-fix-folderoutput-input-field-ui?link_source=email
          nfreese Nowlan Freese made changes -
          Status Reviewing Pull Request [ 10303 ] Merged Needs Testing [ 10002 ]
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ]
          Hide
          ann.loraine Ann Loraine added a comment -

          Deployed for testing to https://bvctest6.bioviz.org.

          Show
          ann.loraine Ann Loraine added a comment - Deployed for testing to https://bvctest6.bioviz.org .
          nfreese Nowlan Freese made changes -
          Status Merged Needs Testing [ 10002 ] Post-merge Testing In Progress [ 10003 ]
          Hide
          nfreese Nowlan Freese added a comment -

          The output folder name field is present, and when the analysis was run the folder was created and named correctly (and the analysis completed correctly).

          Closing issue.

          Show
          nfreese Nowlan Freese added a comment - The output folder name field is present, and when the analysis was run the folder was created and named correctly (and the analysis completed correctly). Closing issue.
          nfreese Nowlan Freese made changes -
          Assignee Nowlan Freese [ nfreese ] Karthik Raveendran [ karthik ]
          nfreese Nowlan Freese made changes -
          Story Points 1 3
          nfreese Nowlan Freese made changes -
          Resolution Done [ 10000 ]
          Status Post-merge Testing In Progress [ 10003 ] Closed [ 6 ]

            People

            • Assignee:
              karthik Karthik Raveendran
              Reporter:
              ann.loraine Ann Loraine
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: