-
Notifications
You must be signed in to change notification settings - Fork 3.6k
bump: try deepspeed >=0.14.1,<=0.15.0
#21076
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
Conversation
⚡ Required checks status: All passing 🟢Groups summary🟢 pytorch_lightning: Tests workflow
These checks are required after the changes to 🟢 pytorch_lightning: Azure GPU
These checks are required after the changes to 🟢 pytorch_lightning: Benchmarks
These checks are required after the changes to 🟢 fabric: Docs
These checks are required after the changes to 🟢 pytorch_lightning: Docs
These checks are required after the changes to 🟢 pytorch_lightning: Docker
These checks are required after the changes to 🟢 lightning_fabric: CPU workflow
These checks are required after the changes to 🟢 lightning_fabric: Azure GPU
These checks are required after the changes to 🟢 mypy
These checks are required after the changes to 🟢 install
These checks are required after the changes to Thank you for your contribution! 💜
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #21076 +/- ##
=======================================
Coverage 87% 87%
=======================================
Files 269 269
Lines 23509 23506 -3
=======================================
- Hits 20494 20492 -2
+ Misses 3015 3014 -1 |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
|
This seems a bit unrelated: |
| # in case you want to preserve/enforce restrictions on the latest compatible version, add "strict" as an in-line comment | ||
|
|
||
| # note: is a bug around 0.10 with `MPS_Accelerator must implement all abstract methods` | ||
| # shall be resolved by https://github.com/microsoft/DeepSpeed/issues/4372 |
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.
Not for this PR, but this was resolved, so maybe we can relax in the future
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.
Yes, I will continue in relaxing. I just wanted to do it in smaller steps so it's safer to land...
* try `deepspeed >=0.14.1,<=0.15.0` * drop from oldest * pip uninstall -y deepspeed * error::DeprecationWarning
…ingModule`, `LightningDataModule` (#21072) * change weights_only default to True * add docs on weights_only arg * add weights_only arg to checkpoint save. weights_only during test set based on ckpt version * add weights_only arg to checkpoint_io * woops, reverting changes * permissions too * fix link * fix another link * datamodule weights_only args * wip: try safe_globals context manager for tests * add weights_only arg to _run_standard_hparams_test * weights_only=False when adding extra_args * switch to lightning_utilities.cli requirements set-oldest (#21077) * bump: try `deepspeed >=0.14.1,<=0.15.0` (#21076) * try `deepspeed >=0.14.1,<=0.15.0` * drop from oldest * pip uninstall -y deepspeed * error::DeprecationWarning * weights_only=True default for torch>=2.6 * changelog * ignore torch.load futurewarning * add .* * will this woork * weights_only according pl version * set env var TORCH_FORCE_NO_WEIGHTS_ONLY_LOAD=1 for pl < 1.5.0 * weights_only=False for omegaconf hparams test * default to weights_only=true for loading from state_dict from url * weights_only=False for hydra * Update src/lightning/fabric/utilities/cloud_io.py Co-authored-by: Jirka Borovec <[email protected]> * defaults for weights_only in torch.hub.load_state_dict_from_url * default to weights_only=False for torch.hub.load_state_dict_from_url * add weights_only to trainer.fit, validate, test, predict * fix tests * add weights_only arg * specify weights_only kwarg * weights_only for fsdp load * Apply suggestions from code review * Apply suggestions from code review * default is none * add weights_only args to strategies * trainer default to weights_only=None * wip: fix typing dump_checkpoint * Apply suggestions from code review * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * weights_only as last arg * asset called with none * weights_only=False for torch>=2.6 in tests * fix changelog description * Empty-Commit * Empty-Commit * trigger ci * skip ddp_fork on macos --------- Co-authored-by: Nicki Skafte Detlefsen <[email protected]> Co-authored-by: Jirka Borovec <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: thomas chaton <[email protected]> Co-authored-by: Jirka B <[email protected]> Co-authored-by: jirka <[email protected]> Co-authored-by: Deependu <[email protected]>
What does this PR do?
'Let''s see if we are fine with this bump
Before submitting
PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:
Reviewer checklist
📚 Documentation preview 📚: https://pytorch-lightning--21076.org.readthedocs.build/en/21076/