Skip to content

Commit

Permalink
remove deprecated use_merged in test
Browse files Browse the repository at this point in the history
  • Loading branch information
echarlaix committed Oct 6, 2023
1 parent a51686e commit e2f8a3b
Showing 1 changed file with 2 additions and 11 deletions.
13 changes: 2 additions & 11 deletions tests/onnxruntime/test_modeling.py
Original file line number Diff line number Diff line change
Expand Up @@ -1958,7 +1958,6 @@ class ORTModelForCausalLMIntegrationTest(ORTModelTestMixin):
FULL_GRID = {
"model_arch": SUPPORTED_ARCHITECTURES,
"use_cache": [False, True],
"use_merged": [False, True],
}

ORTMODEL_CLASS = ORTModelForCausalLM
Expand Down Expand Up @@ -2030,10 +2029,7 @@ def test_merge_from_onnx_and_save(self, model_arch):
self.assertNotIn(ONNX_WEIGHTS_NAME, folder_contents)

@parameterized.expand(grid_parameters(FULL_GRID))
def test_compare_to_transformers(self, test_name: str, model_arch: str, use_cache: bool, use_merged: bool):
if use_cache is False and use_merged is True:
self.skipTest("use_cache=False, use_merged=True are uncompatible")

def test_compare_to_transformers(self, test_name: str, model_arch: str, use_cache: bool):
use_io_binding = None
if use_cache is False:
use_io_binding = False
Expand All @@ -2042,7 +2038,6 @@ def test_compare_to_transformers(self, test_name: str, model_arch: str, use_cach
"test_name": test_name,
"model_arch": model_arch,
"use_cache": use_cache,
"use_merged": use_merged,
}
self._setup(model_args)

Expand Down Expand Up @@ -2100,10 +2095,7 @@ def test_compare_to_transformers(self, test_name: str, model_arch: str, use_cach
gc.collect()

@parameterized.expand(grid_parameters(FULL_GRID))
def test_pipeline_ort_model(self, test_name: str, model_arch: str, use_cache: bool, use_merged: bool):
if use_cache is False and use_merged is True:
self.skipTest("use_cache=False, use_merged=True are uncompatible")

def test_pipeline_ort_model(self, test_name: str, model_arch: str, use_cache: bool):
use_io_binding = None
if use_cache is False:
use_io_binding = False
Expand All @@ -2112,7 +2104,6 @@ def test_pipeline_ort_model(self, test_name: str, model_arch: str, use_cache: bo
"test_name": test_name,
"model_arch": model_arch,
"use_cache": use_cache,
"use_merged": use_merged,
}
self._setup(model_args)

Expand Down

0 comments on commit e2f8a3b

Please sign in to comment.