-
Notifications
You must be signed in to change notification settings - Fork 31
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
Migrate azure/notebooks/HPO-RAPIDS.ipynb to deployment #192
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter 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.
Thanks for copying this over @skirui-source. The build failures were due to a regex bug in the Sphinx extension I wrote, I've pushed another commit to fix that up.
It looks like there are some supporting files related to this notebook like train_rapids.py
and maybe a Dockerfile
? Could you also include those files in the PR?
Also would you mind giving it another pass for spelling/highlighting/etc? I can spot a few places where things like URI is not capetalised, there are spaces missing between words and AmlCompute
isn't code highlighted with backticks.
…oyment into migrate_rapids_hpo_azureml
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.
Thanks for adding the intro. It mentions DGX Cloud so I'm guessing it was copied form the DGX Cloud example notebook. Could you update it to mention Azure ML instead?
Ready to merge. |
Fixes #215 - This PR tracks the migration of rapids_hpo_azureml_notebook to deployment docs.
See #203 (comment) for detailed migration instructions.
Tasks