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

Improve feedback when files can't be opened

    Details

    • Story Points:
      2
    • Sprint:
      Winter 2018 Sprint 3, Spring 2019 Sprint 1

      Description

      Typical users need to open and load data from a variety of file formats. Often, files users try to open have problems that prevent IGB from reading them. Currently, users have a hard time understanding and debugging their files because IGB's error messages upon failure to open or load are confusing or inconsistent.

      Let's implement error-handling code that is consistent across all files types and which provides the user with good information about what when wrong when IGB tries to open a file where the format does not fit the expectation.

        Attachments

          Issue Links

            Activity

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

            Issue:- To provide more informative error message when a file cannot be opened in IGB

            Solution:-
            Provide meaningful message by fetching the root cause from the stack trace and provide a link to IGB help page

            Files changed:-
            core/genometry/src/main/java/com/affymetrix/genometry/util/ErrorHandler.java
            core/igb/src/main/java/com/affymetrix/igb/view/load/GeneralLoadUtils.java
            plugins/bam/src/main/java/org/lorainelab/igb/bam/SAM.java

            Testing steps:-
            (Error files used for testing are attached to the ticket)
            Drag and drop a broken file into IGB get the informative message.

            Test results:-
            Please check the attached test results

            Code changes:-
            https://bitbucket.org/rpatil14/riddhifork-igb/branch/IGBF-1515#diff

            Note:-
            The error message displayed while loading a bed file does not have Arial font and is not in sync with the rest of the application. This issue is fixed with ticket IGBF-1518

            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - - edited Issue:- To provide more informative error message when a file cannot be opened in IGB Solution:- Provide meaningful message by fetching the root cause from the stack trace and provide a link to IGB help page Files changed:- core/genometry/src/main/java/com/affymetrix/genometry/util/ErrorHandler.java core/igb/src/main/java/com/affymetrix/igb/view/load/GeneralLoadUtils.java plugins/bam/src/main/java/org/lorainelab/igb/bam/SAM.java Testing steps:- (Error files used for testing are attached to the ticket) Drag and drop a broken file into IGB get the informative message. Test results:- Please check the attached test results Code changes:- https://bitbucket.org/rpatil14/riddhifork-igb/branch/IGBF-1515#diff Note:- The error message displayed while loading a bed file does not have Arial font and is not in sync with the rest of the application. This issue is fixed with ticket IGBF-1518
            Hide
            svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment -

            I have tested with three attached files. I have checked the code and everything is working as expected.

            Please change the comment in following file and line.
            File : core/genometry/src/main/java/com/affymetrix/genometry/util/ErrorHandler.java
            Line : 130

            Change text from "Add a link to hel page in case of error while loading a file" to "Add a link to help page in case of error while loading a file"

            Show
            svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment - I have tested with three attached files. I have checked the code and everything is working as expected. Please change the comment in following file and line. File : core/genometry/src/main/java/com/affymetrix/genometry/util/ErrorHandler.java Line : 130 Change text from "Add a link to hel page in case of error while loading a file" to "Add a link to help page in case of error while loading a file"
            Hide
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment -

            Corrected spelling error in ErroHandler.java file.

            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Corrected spelling error in ErroHandler.java file.
            Hide
            svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment -

            I have checked the comment. It is now corrected.

            Show
            svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment - I have checked the comment. It is now corrected.
            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Pull request submitted:- https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/666/igbf-1515-improve-feedback-when-files-cant/diff
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged into master.

            Show
            ann.loraine Ann Loraine added a comment - Merged into master.
            Hide
            karthik Karthik Raveendran added a comment -

            Testing Windows: Working correctly

            Show
            karthik Karthik Raveendran added a comment - Testing Windows: Working correctly
            Hide
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment -

            Windows testing successful

            Show
            rpatil14 Riddhi Jagdish Patil (Inactive) added a comment - Windows testing successful
            Hide
            Jill Jill Jenkins (Inactive) added a comment -

            MacOS testing successful.

            Show
            Jill Jill Jenkins (Inactive) added a comment - MacOS testing successful.
            Hide
            svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment -

            Linux testing successful.

            Show
            svallapu Sai Charan Reddy Vallapureddy (Inactive) added a comment - Linux testing successful.
            Show
            nfreese Nowlan Freese added a comment - I made a few changes to the error formatting: https://bitbucket.org/lorainelab/integrated-genome-browser/pull-requests/757/igbf-1515-2-additional-formatting-of-file/diff
            Hide
            ann.loraine Ann Loraine added a comment -

            Merged but not yet deployed to "early releases" on Bioviz.

            Show
            ann.loraine Ann Loraine added a comment - Merged but not yet deployed to "early releases" on Bioviz.

              People

              • Assignee:
                rpatil14 Riddhi Jagdish Patil (Inactive)
                Reporter:
                ann.loraine Ann Loraine
              • Votes:
                0 Vote for this issue
                Watchers:
                6 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: