LudovicoYIN commented on code in PR #18670:
URL: https://github.com/apache/tvm/pull/18670#discussion_r2710631860
##########
tests/python/relax/test_frontend_from_exported_program.py:
##########
@@ -8787,5 +8787,33 @@ def main(
verify_model(UpsampleNearest2dSize(), example_args, {}, expected_size)
+def test_from_exported_program_sparse_csr_buffer():
+ if not hasattr(torch, "sparse_csr"):
+ pytest.skip("sparse CSR tensor is not supported in this PyTorch build")
Review Comment:
My intent was to run this test only on PyTorch builds that actually support
torch.sparse_csr, to avoid false failures on builds without CSR support. In my
local environment the test still passes even if I remove the skip, but I added
it for portability. Do you prefer keeping the skip for compatibility, or should
I remove it?
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
