Skip to content

Commit

Permalink
disable flaky tests (#1629)
Browse files Browse the repository at this point in the history
  • Loading branch information
t-vi authored Jan 10, 2025
1 parent 19f3d5c commit 6708b16
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 22 deletions.
4 changes: 0 additions & 4 deletions thunder/tests/test_dynamo.py
Original file line number Diff line number Diff line change
Expand Up @@ -445,10 +445,6 @@ def func(x):
IS_WINDOWS,
reason="torch.compile Windows support is still WIP - https://github.com/pytorch/pytorch/issues/122094",
),
pytest.mark.skipif(
version_between(torch.__version__, min_ver="2.6.0dev0", max_ver="2.6.0a99"),
reason="https://github.com/Lightning-AI/lightning-thunder/issues/1471",
),
),
)
@requiresCUDA
Expand Down
26 changes: 12 additions & 14 deletions thunder/tests/test_networks.py
Original file line number Diff line number Diff line change
Expand Up @@ -122,8 +122,16 @@ def test_nanogpt_complete_cudagraphs(executor, device, dtype):
assert _there_is_cudagraph_sym(thunder.last_traces(tom)[-1])


@instantiate(dtypes=(thunder.float32,), devicetypes=(thunder.devices.DeviceType.CUDA,))
@requiresCUDA
@instantiate(
dtypes=(thunder.float32,),
devicetypes=(thunder.devices.DeviceType.CUDA,),
decorators=(
pytest.mark.skipif(
version_between(torch.__version__, min_ver="2.7.0dev0", max_ver="2.7.0a99"),
reason="https://github.com/lightning-ai/lightning-thunder/pull/1629",
),
),
)
def test_nanogpt_complete_cudagraphs_autograd(executor, device, dtype):
tdtype = ttorch.to_torch_dtype(dtype)

Expand Down Expand Up @@ -224,12 +232,6 @@ def test_nanogpt_mlp(executor, device, dtype):
@instantiate(
dtypes=(thunder.float32,),
executors=all_test_executors_and_dynamo,
decorators=(
pytest.mark.skipif(
version_between(torch.__version__, min_ver="2.6.0dev0", max_ver="2.6.0a99"),
reason="https://github.com/Lightning-AI/lightning-thunder/issues/1471",
),
),
)
def test_nanogpt_gelu(executor, device, dtype):
tdtype = ttorch.to_torch_dtype(dtype)
Expand Down Expand Up @@ -366,8 +368,8 @@ def test_quantization():


@pytest.mark.skipif(
version_between(torch.__version__, min_ver="2.6.0dev0", max_ver="2.6.0a99"),
reason="https://github.com/Lightning-AI/lightning-thunder/issues/1471",
version_between(torch.__version__, min_ver="2.7.0dev0", max_ver="2.7.0a99"),
reason="https://github.com/bitsandbytes-foundation/bitsandbytes/pull/1629",
)
@thunder.tests.framework.requiresCUDA
def test_thunderfx_mistral_nemo_small():
Expand Down Expand Up @@ -420,10 +422,6 @@ def test_thunderfx_mistral_nemo_small():


# disabled "Qwen/Qwen2.5-7B-Instruct" see https://github.com/NVIDIA/Fuser/issues/3682
@pytest.mark.skipif(
version_between(torch.__version__, min_ver="2.6.0dev0", max_ver="2.6.0a99"),
reason="https://github.com/Lightning-AI/lightning-thunder/issues/1471",
)
@thunder.tests.framework.requiresCUDA
@pytest.mark.parametrize("model_id", ["microsoft/Phi-3-mini-128k-instruct"])
def test_hf_for_nemo(model_id):
Expand Down
4 changes: 0 additions & 4 deletions thunder/tests/test_recipes.py
Original file line number Diff line number Diff line change
Expand Up @@ -24,10 +24,6 @@ def test_recipe_basic_bert():
assert_close(actual, expected)


@pytest.mark.skipif(
version_between(torch.__version__, min_ver="2.6.0dev0", max_ver="2.6.0a99"),
reason="https://github.com/Lightning-AI/lightning-thunder/issues/1471",
)
def test_recipe_basic_bert_dynamo():
bert = transformers.BertForSequenceClassification(transformers.BertConfig())
del bert.bert.encoder.layer[1:]
Expand Down

0 comments on commit 6708b16

Please sign in to comment.