farshidsp commented on code in PR #12504: URL: https://github.com/apache/tvm/pull/12504#discussion_r953130711
########## python/tvm/contrib/hexagon/pytest_plugin.py: ########## @@ -56,13 +56,16 @@ def _compose(args, decs): requires_hexagon_toolchain = tvm.testing.requires_hexagon(support_required="compile-only") [email protected](scope="session") -def android_serial_number() -> Optional[str]: Review Comment: Since `pytest_configure(config)` is a pytest hook, there were no easy way to have it use a pytest fixtures. Maybe I'm missing the point, but I did not see the usefulness of having the android_serial_number as a` pytest.fixture` and by removing it, it solved the previous error. -- 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]
