Skip to content

Update notebooks with dockstore info #25

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Nov 9, 2022

Conversation

anilnatha
Copy link
Collaborator

The changes in this pull request are in preparation for @rtapella demo during the Unity status meeting.

  1. Updated Application notebook with detailed information we learned during Nick's Dockstore walkthrough.
  2. Updated Jobs notebook so that we now show how to query dockstore, select an application from the results, download a zip containing the selected application's metadata, and a placeholder cell which will ultimately be used to deploy the application to SPS.

Let's review this in the morning ASAP as I have meetings starting at 9 and won't be out of them until 12.

…ion, downloading a zip containing application metadata, and a placeholder cell where we will package up an application for deployment to SPS. Also renumbered subsequent cells.
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@anilnatha
Copy link
Collaborator Author

@rtapella Made some minor improvements to the applications notebook during the Sounder SIPS meeting. Just letting you know so you can pull them down, if needed.

@riverma
Copy link
Contributor

riverma commented Nov 8, 2022

Hey @anilnatha - this might be a good repo to add some of the template-repo file essentials (e.g. pull request template text). Mind if I send out some PRs for the file content on this repo?

Copy link
Collaborator

@rtapella rtapella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good version 1

@rtapella rtapella merged commit e082690 into develop Nov 9, 2022
@rtapella rtapella deleted the update-notebooks-with-dockstore-info branch November 9, 2022 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants