comaniac commented on a change in pull request #8542:
URL: https://github.com/apache/tvm/pull/8542#discussion_r675723225
##########
File path: python/tvm/testing.py
##########
@@ -795,15 +795,32 @@ def _auto_parametrize_target(metafunc):
"""
if "target" in metafunc.fixturenames:
- parametrized_args = [
- arg.strip()
- for mark in metafunc.definition.iter_markers("parametrize")
- for arg in mark.args[0].split(",")
- ]
-
- if "target" not in parametrized_args:
- # Check if the function is marked with either excluded or
- # known failing targets.
+ for mark in metafunc.definition.iter_markers("parametrize"):
Review comment:
nit: Better not to use loop-else construct (e.g.,
https://intoli.com/blog/for-else-in-python/).
--
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]
