gromero commented on a change in pull request #8331:
URL: https://github.com/apache/tvm/pull/8331#discussion_r658302581
##########
File path: tests/python/driver/tvmc/conftest.py
##########
@@ -167,40 +148,17 @@ def onnx_mnist():
return model_file
[email protected](scope="session")
-def tflite_compiled_model(tmpdir_factory):
-
- # Not all CI environments will have TFLite installed
- # so we need to safely skip this fixture that will
- # crash the tests that rely on it.
- # As this is a pytest.fixture, we cannot take advantage
- # of pytest.importorskip. Using the block below instead.
- try:
- import tflite
- except ImportError:
- print("Cannot import tflite, which is required by
tflite_compiled_module_as_tarfile.")
- return ""
-
- target_dir = tmpdir_factory.mktemp("data")
- return get_sample_compiled_module(target_dir, "mock.tar")
-
-
[email protected](scope="session")
-def tflite_compiled_model_mlf(tmpdir_factory):
[email protected]
+def tflite_tvmc_compiler(tmpdir_factory):
Review comment:
How about renaming the `tflite_tvmc_compiler` fixture to
`tflite_compile_model`, just because I don't think that `tvmc` in there is
informative. But no strong feelings about it. Feel free also to get more input
from Leandro before sending a v2.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
