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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: Moving trainium from Blueprints to GenAI #501

Closed
wants to merge 2 commits into from

Conversation

sheetaljoshi
Copy link

@sheetaljoshi sheetaljoshi commented Apr 9, 2024

What does this PR do?

馃洃 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Trainium on EKS is better suited to be under Gen AI -> Training -> BERT-Large on Trainium section. I recommend moving this content under Gen AI for better indexing, search ability, as well as avoiding duplication of the content.

Motivation

It make sense to move the training under Gen AI for better indexing and searchability.

More

  • Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
  • Mandatory for new blueprints. Yes, I have added a example to support my blueprint PR
  • Mandatory for new blueprints. Yes, I have updated the website/docs or website/blog section for this feature
  • Yes, I ran pre-commit run -a with this PR. Link for installing pre-commit locally

For Moderators

  • E2E Test successfully complete before merge?

Additional Notes

@sheetaljoshi sheetaljoshi changed the title Moving trainium from Blueprints to GenAI docs: Moving trainium from Blueprints to GenAI Apr 9, 2024
@vara-bonthu
Copy link
Contributor

Thanks for the PR @sheetaljoshi ! Let's retain the original Trainium.md document for deploying the Trainium blueprint which hosts number of Gen AI blueprints.

We'll transfer only the BERT-Large Training on Trainium example blueprint deployment to the GEN AI page. The BERT Large example blueprint will reference Trainium.md as a prerequisite.

Copy link
Contributor

This PR has been automatically marked as stale because it has been open 30 days
with no activity. Remove stale label or comment or this PR will be closed in 10 days

@github-actions github-actions bot added the stale label May 10, 2024
Copy link
Contributor

Pull request closed due to inactivity.

@github-actions github-actions bot closed this May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants