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

Sage maker tutorial #35

Merged
merged 6 commits into from
Sep 18, 2024
Merged

Sage maker tutorial #35

merged 6 commits into from
Sep 18, 2024

Conversation

LoganColeNIH
Copy link

@LoganColeNIH LoganColeNIH commented Sep 17, 2024

Pull Request Template

Description

Extended SageMaker tutorial to include idle shutdown configuration and SageMaker Studio.

Assignee

Assignees: @LoganColeNIH

PR checklist

Please ensure the following:

  • This comment contains a description of changes (with reason).
  • All changes were tested.
  • If you've fixed a bug mention the issue number/name.
  • Apply approriate tags (e.g. documentation, bug)

@trimbleNIH
Copy link
Contributor

@JohnTurnerNIH , Looks like this is hitting the link-checker error too. After that's fixed, we can test it again.

@JohnTurnerNIH
Copy link
Contributor

@JohnTurnerNIH , Looks like this is hitting the link-checker error too. After that's fixed, we can test it again.

Question is, do we merge this with known bad links and fix in another PR or wait until all are fixed with this PR. Since this PR came before the link checking I say we make it another PR

@kyleoconnell-NIH kyleoconnell-NIH merged commit 28d499f into main Sep 18, 2024
1 of 3 checks passed
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.

4 participants