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

Fix torch check #6402

Merged
merged 1 commit into from
Aug 22, 2024
Merged

Fix torch check #6402

merged 1 commit into from
Aug 22, 2024

Conversation

loadams
Copy link
Contributor

@loadams loadams commented Aug 22, 2024

No description provided.

@loadams loadams merged commit 55b4cae into master Aug 22, 2024
10 of 11 checks passed
@loadams loadams deleted the loadams/fix-torch-gds branch August 22, 2024 22:46
loadams added a commit that referenced this pull request Aug 27, 2024
loadams added a commit that referenced this pull request Aug 27, 2024
loadams added a commit that referenced this pull request Aug 27, 2024
loadams added a commit that referenced this pull request Aug 27, 2024
loadams added a commit that referenced this pull request Sep 27, 2024
…ch workflow triggers (#6584)

Changes from #6472 caused the no-torch workflow that is an example of
how we build the DeepSpeed release package to fail (so we caught this
before a release, see more in #6402). These changes also copy the style
used to include torch in other accelerator op_builder implementations,
such as npu
[here](https://github.com/microsoft/DeepSpeed/blob/master/op_builder/npu/fused_adam.py#L8)
and hpu
[here](https://github.com/microsoft/DeepSpeed/blob/828ddfbbda2482412fffc89f5fcd3b0d0eba9a62/op_builder/hpu/fused_adam.py#L15).

This also updates the no-torch workflow to run on all changes to the
op_builder directory. The test runs quickly and shouldn't add any
additional testing burden there.

Resolves: #6576
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.

1 participant