Skip to content

Conversation

@pacman100
Copy link
Contributor

@pacman100 pacman100 commented Jul 21, 2022

What does this PR do?

  1. DeepSpeed integration is broken when using the latest deepspeed 0.6.7 version. This PR fixes it.

Tracking in accelerate with issue #536

@pacman100 pacman100 requested review from muellerzr and sgugger July 21, 2022 12:31
Copy link
Contributor

@muellerzr muellerzr left a comment

Choose a reason for hiding this comment

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

Thanks! Might be worth keeping an issue open about this until its resolved to send people to as they get issues/they can find it easily

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Jul 21, 2022

The documentation is not available anymore as the PR was closed or merged.

Copy link
Collaborator

@sgugger sgugger left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Collaborator

@sgugger sgugger left a comment

Choose a reason for hiding this comment

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

Nice to actually fix the root of the issue!

Copy link
Contributor

@muellerzr muellerzr left a comment

Choose a reason for hiding this comment

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

Nice fix!

@pacman100 pacman100 changed the title deepspeed version hotfix deepspeed version 0.6.7 fix Jul 22, 2022
@pacman100 pacman100 merged commit e98dc22 into huggingface:main Jul 22, 2022
@pacman100 pacman100 deleted the smangrul/deepspeed-test-hotfix branch July 27, 2022 04:20
@pacman100 pacman100 mentioned this pull request Mar 2, 2023
1 task
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