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

Unpin deepspeed #3466

Merged
merged 3 commits into from
Jul 14, 2023
Merged

Unpin deepspeed #3466

merged 3 commits into from
Jul 14, 2023

Conversation

arnavgarg1
Copy link
Contributor

Previously, we were defaulting to 0.8.3 which has a few bugs that are fixed in deepspeed 0.9.2. I'm temporarily pinning this version, and will test deepspeed 0.9.5 at a future point so we can unpin deepspeed.

@arnavgarg1 arnavgarg1 changed the title temporarily pin deepspeed to 0.9.2 Unpin deepspeed Jul 13, 2023
@github-actions
Copy link

Unit Test Results

  4 files  ±0    4 suites  ±0   54m 59s ⏱️ - 2m 29s
34 tests ±0  29 ✔️ ±0    5 💤 ±0  0 ±0 
68 runs  ±0  58 ✔️ ±0  10 💤 ±0  0 ±0 

Results for commit e527c97. ± Comparison against base commit 5c92f62.

@arnavgarg1 arnavgarg1 merged commit 12dd12e into master Jul 14, 2023
14 of 16 checks passed
@arnavgarg1 arnavgarg1 deleted the bump_deepspeed branch July 14, 2023 00:24
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.

None yet

2 participants