Skip to content

Commit

Permalink
Add back external colossalai test (#16817)
Browse files Browse the repository at this point in the history
  • Loading branch information
justusschock authored Feb 20, 2023
1 parent 60004eb commit c7962a1
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion requirements/pytorch/strategies.txt
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
# in case you want to preserve/enforce restrictions on the latest compatible version, add "strict" as an in-line comment

deepspeed>=0.6.0, <0.8.0 # TODO: Include 0.8.x after https://github.com/microsoft/DeepSpeed/commit/b587c7e85470329ac25df7c7c2521ff9b2833db7 gets released
lightning-colossalai==0.1.0dev
lightning-colossalai==0.1.0dev1
Original file line number Diff line number Diff line change
Expand Up @@ -841,7 +841,6 @@ def get_defaults(cls):

@RunIf(min_cuda_gpus=1) # trigger this test on our GPU pipeline, because we don't install the package on the CPU suite
@pytest.mark.skipif(not package_available("lightning_colossalai"), reason="Requires Colossal AI Strategy")
@pytest.mark.skip
def test_colossalai_external_strategy(monkeypatch):
with mock.patch(
"lightning.pytorch.trainer.connectors.accelerator_connector._LIGHTNING_COLOSSALAI_AVAILABLE", False
Expand Down

0 comments on commit c7962a1

Please sign in to comment.