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.

Sign up for GitHub

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

Jump to bottom

Add troubleshooting section in README and improvements to training_example.py #192

Merged
merged 29 commits into from
Feb 12, 2024
Merged

Add troubleshooting section in README and improvements to training_example.py #192

merged 29 commits into from
Feb 12, 2024

Conversation

sfloresk
Copy link
Contributor

@sfloresk sfloresk commented Feb 8, 2024

Description

  • Add metadata options to main.tf
  • Remove unnecessary code from training_example.py
  • Add checkpoint persistence in S3 in training_example.py
  • Add troubleshooting section to README.md

Motivation and Context

Improve distributed-machine-learning blueprint docs and training script

How Has This Been Tested?

  • I have tested and validated these changes using one or more of the provided examples/* projects
  • I have executed pre-commit run -a on my pull request

sfloresk and others added 24 commits December 12, 2023 07:51
Add distributed-ml-training blueprint
1f74126
Fix docs
3ebc805
Fix docs
d15e674
Fix format
20f7587
Add solution diagram and reference links
3148419
Add solution diagram and reference links
2d7bdbe
Fix typo in docs
29a20e6
@sfloresk
Merge branch 'main' into main
fbadf11
Add training example file and simplify docs
e372d83
Add training example script file
6ccd034
Update task to use read only root fs
35cf696
@sfloresk
Remove EFS - Add S3
ecc44f3
@sfloresk
Merge branch 'main' of https://github.com/sfloresk/ecs-blueprints
9c148b0
@sfloresk
Fix bugs and remove deployment of supporting resources
a9b4436
@sfloresk
Change aws provider version to >= 5.0
4001305
@sfloresk
Improve docs format
49f5c07
@sfloresk
Include region in test commands, change output bucket arn to id
486dd47
@sfloresk
Merge branch 'main' into main
70500d1
@sfloresk
Change bucket ARN for name in the docs
a32c91d
@sfloresk
Merge branch 'main' of https://github.com/sfloresk/ecs-blueprints
16daf47
@sfloresk
Add result of training script
da408a8
@sfloresk
fix README type and enforce metadata service to v2
78c5fca
@sfloresk
Merge branch 'main' into main
7d13157
@sfloresk
Add troubleshooting section to README. Add checkpoint and remove shar…
e7f9bb4 
…ed storage refs in training script
@sfloresk sfloresk requested a review from a team as a code owner February 8, 2024 21:35
@joozero joozero merged commit 730a775 into aws-ia:main Feb 12, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@joozero joozero joozero approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sfloresk @joozero