-
Notifications
You must be signed in to change notification settings - Fork 2
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 dependency deepspeed to v0.15.1 - autoclosed #32
Update dependency deepspeed to v0.15.1 - autoclosed #32
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
WalkthroughThe update in the Changes
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
poetry.lock
is excluded by!**/*.lock
Files selected for processing (1)
- pyproject.toml (1 hunks)
Files skipped from review due to trivial changes (1)
- pyproject.toml
1f3eb57
to
2588c48
Compare
2588c48
to
0e2963f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
poetry.lock
is excluded by!**/*.lock
Files selected for processing (1)
- pyproject.toml (1 hunks)
Additional comments not posted (1)
pyproject.toml (1)
24-24
: Dependency update approved.The update of the
deepspeed
dependency to version0.14.5
is consistent with the PR objectives and should enhance the project's performance and capabilities.
0818d3d
to
f36064c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files ignored due to path filters (1)
poetry.lock
is excluded by!**/*.lock
Files selected for processing (1)
- pyproject.toml (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- pyproject.toml
f36064c
to
43e783c
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
43e783c
to
27fd2c7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (1)
pyproject.toml (1)
24-24
: Potential Compatibility Issues withdeepspeed
Version UpdateThe update of the
deepspeed
package to version0.15.1
inpyproject.toml
may introduce compatibility issues due to existing constraints and specific usages in the codebase:
- Dependency Constraints: The
poetry.lock
file indicates constraints such asdeepspeed (<=0.14.0)
, which conflict with the new version0.15.1
.- Codebase Usage: Several scripts and files, such as
train_sft.sh
,train_rm.sh
, andload_tokenizer.py
, usedeepspeed
and may require adjustments to work with the updated version.Please review and update the
poetry.lock
file to resolve any conflicts and test the affected scripts and functionalities to ensure compatibility withdeepspeed
version0.15.1
.Analysis chain
Version update approved.
The update of the
deepspeed
package from0.10.2
to0.15.1
is correctly implemented in thepyproject.toml
file. This change is expected to bring new features and improvements as described in the PR.Please ensure to verify the compatibility of the new
deepspeed
version with the existing project code and other dependencies. This can involve running existing test suites or setting up new tests to cover the changes introduced by the update.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify compatibility of deepspeed v0.15.1 with the project. # Test: Run project's test suite. Expect: All tests pass without errors. echo "Run the project's test suite to ensure compatibility."Length of output: 119
Script:
#!/bin/bash # Search for any usage of the deepspeed package in the codebase to identify potential compatibility issues. rg 'deepspeed' -A 5 # Check for any dependency conflicts in the pyproject.toml file. # This assumes that the file might have other dependencies that could conflict with the new version of deepspeed. rg 'version = ".*"' pyproject.toml -A 2Length of output: 10277
This PR contains the following updates:
0.10.2
->0.15.1
Release Notes
microsoft/DeepSpeed (deepspeed)
v0.15.1
: Patch releaseCompare Source
What's Changed
CUDA_HOME
is not defined on ROCm systems by @amorehead in https://github.com/microsoft/DeepSpeed/pull/6488New Contributors
Full Changelog: microsoft/DeepSpeed@v0.15.0...v0.15.1
v0.15.0
: DeepSpeed v0.15.0Compare Source
What's Changed
New Contributors
Full Changelog: microsoft/DeepSpeed@v0.14.5...v0.15.0
v0.14.5
: Patch releaseCompare Source
What's Changed
warned
by @anferico in https://github.com/microsoft/DeepSpeed/pull/5725torch.nan_to_num
replace numpy wrapper one by @jinyouzhi in https://github.com/microsoft/DeepSpeed/pull/5877New Contributors
Full Changelog: microsoft/DeepSpeed@v0.14.4...v0.14.5
v0.14.4
: Patch releaseCompare Source
What's Changed
New Contributors
Full Changelog: microsoft/DeepSpeed@v0.14.3...v0.14.4
v0.14.3
: Patch releaseCompare Source
What's Changed
New Contributors
Full Changelog: microsoft/DeepSpeed@v0.14.2...v0.14.3
v0.14.2
: Patch releaseCompare Source
What's Changed
split_index
method by @bm-synth in https://github.com/microsoft/DeepSpeed/pull/5292DistributedDataAnalyzer
map-reduce by @bm-synth in https://github.com/microsoft/DeepSpeed/pull/5291New Contributors
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.