-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
Marketplace - Sample notebook templates for sellers #1330
Conversation
Check out this pull request on Review Jupyter notebook visual diffs & provide feedback on notebooks. Powered by ReviewNB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple things stand out:
- spaces in the path
- special characters in the file name "<" and ">"
Hi Aaron, I have addressed these two issues. |
...and_sample_notebook/Algorithm/Sample_Notebook_Template/title_of_your_product-Algorithm.ipynb
Outdated
Show resolved
Hide resolved
...and_sample_notebook/Algorithm/Sample_Notebook_Template/title_of_your_product-Algorithm.ipynb
Show resolved
Hide resolved
...and_sample_notebook/Algorithm/Sample_Notebook_Template/title_of_your_product-Algorithm.ipynb
Outdated
Show resolved
Hide resolved
...and_sample_notebook/Algorithm/Sample_Notebook_Template/title_of_your_product-Algorithm.ipynb
Outdated
Show resolved
Hide resolved
...and_sample_notebook/Algorithm/Sample_Notebook_Template/title_of_your_product-Algorithm.ipynb
Outdated
Show resolved
Hide resolved
Incorporating suggestions by @aaronmarkham Co-authored-by: Aaron Markham <[email protected]>
@aaronmarkham I have incorporated your and Kate's review comments. |
* For sellers : Algorithms and ModelPackages sample notebook templates * fixing cosmetic issue * Renaming files, removing spaces in path * renaming files and folders * removing Excel and adding a markdown for describing listing requirements * Apply suggestions from code review Incorporating suggestions by @aaronmarkham Co-authored-by: Aaron Markham <[email protected]> * incorporating cosmetic changes suggested by @aaronmarkham * changing the AWS Marketplace reference in documentation * Fixing sample notebook TOC * incorporating Kate's review comments Co-authored-by: Waikar <[email protected]> Co-authored-by: Aaron Markham <[email protected]>
Issue #, if available: N/A
Description of changes:
I would like to release sample notebook templates which AWS Marketplace sellers can use to craft the sample notebooks to be associated with their AWS Marketplace listings
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.