Skip to content

[Tests] Skip source-tree-only tests in wheel testing - #19705

Closed
tlopex wants to merge 2 commits into
apache:mainfrom
tlopex:fix-wheel-source-tree-only-tests
Closed

[Tests] Skip source-tree-only tests in wheel testing#19705
tlopex wants to merge 2 commits into
apache:mainfrom
tlopex:fix-wheel-source-tree-only-tests

Conversation

@tlopex

@tlopex tlopex commented Jun 9, 2026

Copy link
Copy Markdown
Member

This pr skip tests that require a source checkout when running against the Python wheel.

The apache-tvm==0.25.0rc0 wheel does not include the source/include tree needed by tvm.libinfo.find_include_path(). Some tests call export_library() or JIT through the C host backend, which then fails in wheel testing with:

AssertionError: Cannot find the source directory given ffi_dir: .../site-packages/tvm

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates several test files in the codegen and relax test suites to skip specific tests during wheel testing. This is achieved by adding the @tvm.testing.skip_if_wheel_test decorator to individual test functions and parameterizing the 'c' target with the skip mark. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@tlopex tlopex changed the title [Test] Skip source-tree-only tests in wheel testing [Tests] Skip source-tree-only tests in wheel testing Jun 9, 2026
@tqchen tqchen closed this Jun 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants