Skip to content

Commit

Permalink
Bump deepspeed 20240727 (#1790)
Browse files Browse the repository at this point in the history
* pin deepspeed to 0.14.4 otherwise it doesn't play nice with trl

* Add test to import to try to trigger import dependencies
  • Loading branch information
winglian authored Jul 27, 2024
1 parent 6a9cfec commit 2268091
Show file tree
Hide file tree
Showing 3 changed files with 22 additions and 2 deletions.
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ transformers==4.43.1
tokenizers==0.19.1
bitsandbytes==0.43.1
accelerate==0.32.0
deepspeed @ git+https://github.com/microsoft/DeepSpeed.git@bc48371c5e1fb8fd70fc79285e66201dbb65679b
deepspeed==0.14.4
pydantic==2.6.3
addict
fire
Expand Down
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ def parse_requirements():
"fused-dense-lib @ git+https://github.com/Dao-AILab/[email protected]#subdirectory=csrc/fused_dense_lib",
],
"deepspeed": [
"deepspeed @ git+https://github.com/microsoft/DeepSpeed.git@bc48371c5e1fb8fd70fc79285e66201dbb65679b",
"deepspeed==0.14.4",
"deepspeed-kernels",
],
"mamba-ssm": [
Expand Down
20 changes: 20 additions & 0 deletions tests/e2e/test_imports.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
"""
test module to import various submodules that have historically broken due to dependency issues
"""
import unittest


class TestImports(unittest.TestCase):
"""
Test class to import various submodules that have historically broken due to dependency issues
"""

def test_import_causal_trainer(self):
from axolotl.core.trainer_builder import ( # pylint: disable=unused-import # noqa: F401
HFCausalTrainerBuilder,
)

def test_import_rl_trainer(self):
from axolotl.core.trainer_builder import ( # pylint: disable=unused-import # noqa: F401
HFRLTrainerBuilder,
)

0 comments on commit 2268091

Please sign in to comment.