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 IPEX MultiNode Docs #228

Merged
merged 1 commit into from
Jul 8, 2024

Conversation

tylertitsworth
Copy link
Contributor

Description

Add the Deepspeed BKM and reflect the latest image updates.

Related Issue

MLOPS-2007

Changes Made

  • Update multinode image tables
  • Fix some grammar/typos
  • Consolidate port-based BKM
  • add python snippets for DDP init
  • add Deepspeed BKM section
  • The code follows the project's coding standards.
  • No Intel Internal IP is present within the changes.
  • The documentation has been updated to reflect any changes in functionality.

Validation

n/a

  • I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

Signed-off-by: tylertitsworth <[email protected]>
@tylertitsworth tylertitsworth added documentation Improvements or additions to documentation Review labels Jul 8, 2024
@tylertitsworth tylertitsworth self-assigned this Jul 8, 2024
Copy link
Contributor

@sramakintel sramakintel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this might look a little weird, but this is accurate, deepspeed isn't compatible with certain versions of torch + idp

@tylertitsworth tylertitsworth enabled auto-merge (squash) July 8, 2024 23:43
Copy link

github-actions bot commented Jul 8, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

@tylertitsworth tylertitsworth merged commit e59f2e6 into main Jul 8, 2024
22 checks passed
@tylertitsworth tylertitsworth deleted the tylertitsworth/update-ipex-mult-docs branch July 8, 2024 23:48
dmsuehir pushed a commit that referenced this pull request Jul 12, 2024
Signed-off-by: tylertitsworth <[email protected]>
Signed-off-by: Dina Suehiro Jones <[email protected]>
jitendra42 pushed a commit to jitendra42/ai-containers that referenced this pull request Oct 23, 2024
* rename things

* Bump actions/upload-artifact from 3 to 4

Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 3 to 4.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v3...v4)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: tylertitsworth <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
jitendra42 pushed a commit to jitendra42/ai-containers that referenced this pull request Oct 23, 2024
Revert "Bump actions/upload-artifact from 3 to 4 (intel#228)"

This reverts commit 37056f7.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants