Details

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

      Attachments

        Issue Links

          Activity

          Hide
          pkulzer Paige Kulzer (Inactive) added a comment -

          Tested the Downsampler Filter on Mac using IGB 10.2.0. The app is available from the App Manager and installed with no errors. The app also appears in the IGB App Store with the correct version number (0.0.3). I'm able to use the app to filter reads within a BAM file track. The instructions in the app description are clear and easy to follow. Based on the above comments, I made sure that the app's JAR files were present in the Downloads section on BitBucket. Similarly, I checked that the logo is present in the repo, as well.

          All looks good to me, so I'm closing this ticket now.

          Show
          pkulzer Paige Kulzer (Inactive) added a comment - Tested the Downsampler Filter on Mac using IGB 10.2.0. The app is available from the App Manager and installed with no errors. The app also appears in the IGB App Store with the correct version number (0.0.3). I'm able to use the app to filter reads within a BAM file track. The instructions in the app description are clear and easy to follow. Based on the above comments, I made sure that the app's JAR files were present in the Downloads section on BitBucket. Similarly, I checked that the logo is present in the repo, as well. All looks good to me, so I'm closing this ticket now.
          Hide
          karthik Karthik Raveendran added a comment -

          Resubmitted the app with the requested changes.

          Show
          karthik Karthik Raveendran added a comment - Resubmitted the app with the requested changes.
          Hide
          ann.loraine Ann Loraine added a comment -

          Thank you Karthik Raveendran! I updated the Loraine Lab fork with your new changes.

          One last request !

          Since the README.md is basically a copy of the AppDescription.md text, how about changing the POM.xml so that it uses the README.md for the "app description" variable? Then we won't have to maintain two different files, and you can delete the AppDescription.md.

          Show
          ann.loraine Ann Loraine added a comment - Thank you Karthik Raveendran ! I updated the Loraine Lab fork with your new changes. One last request ! Since the README.md is basically a copy of the AppDescription.md text, how about changing the POM.xml so that it uses the README.md for the "app description" variable? Then we won't have to maintain two different files, and you can delete the AppDescription.md.
          Hide
          karthik Karthik Raveendran added a comment - - edited

          The 3 changes requested above has been implemented in the repo .

          Show
          karthik Karthik Raveendran added a comment - - edited The 3 changes requested above has been implemented in the repo .
          Hide
          ann.loraine Ann Loraine added a comment - - edited

          Change requests:

          Show
          ann.loraine Ann Loraine added a comment - - edited Change requests: Add the image you created as a new file in the repository - do it like this: https://bitbucket.org/lorainelab/app-for-testing-app-workflow/src/testing/src/main/resources/ Within Bitbucket, change the repository settings so that it uses the image as the repository logo / avatar Create a better "README.md" for the repository (see: https://bitbucket.org/KarthikRavee91/downsampler_filter/src/main/ )

            People

            • Assignee:
              karthik Karthik Raveendran
              Reporter:
              karthik Karthik Raveendran
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: