Skip to content
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

Update execute notebook on startup script #83

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

philipskokoh
Copy link

  • Current SageMaker Notebook instance does not have pytorch_p36. It's pytorch_p37 now.
  • The current nbconvert version requires explicit --to parameter.

Issue #, if available:
The script is not working, and needs some modification.

Description of changes:
Few modification to make the lifecycle configuration script work on the current SageMaker Notebook instance.

  • Current SageMaker Notebook instance does not have pytorch_p36. It's pytorch_p37 now.
  • The current nbconvert version requires explicit --to parameter.

Testing Done

  • Notebook Instance created successfully with the Lifecycle Configuration
  • Notebook Instance stopped and started successfully
  • Documentation in the script around any network access requirements
  • Documentation in the script around any IAM permission requirements
  • CLI commands used to validate functionality on the instance
  • New script link and description added to README.md
# Provide your commands here
/you/commands/here

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

- Current SageMaker Notebook instance does not have `pytorch_p36`. It's `pytorch_p37` now. 
- The current `nbconvert` version requires explicit `--to` parameter.
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.

None yet

1 participant