Skip to content

Report backend opt-outs as skipped#99

Merged
andife merged 7 commits into
onnx:mainfrom
emmtrix:codex/tvm-skip-reporting
Mar 20, 2026
Merged

Report backend opt-outs as skipped#99
andife merged 7 commits into
onnx:mainfrom
emmtrix:codex/tvm-skip-reporting

Conversation

@strimo378

Copy link
Copy Markdown
Collaborator
  • record backend opt-out exceptions as real pytest skips so they appear under skipped in report.json and trend.json
  • this will dramatically change the scoreboard results

@strimo378 strimo378 assigned andife and unassigned andife Mar 20, 2026
@strimo378 strimo378 requested a review from andife March 20, 2026 00:30
@andife

andife commented Mar 20, 2026

Copy link
Copy Markdown
Member

@strimo378 Are the changes compatible with the historical data? Sure, the curves dip, but otherwise the other data points should remain intact.

@andife andife closed this Mar 20, 2026
@andife andife reopened this Mar 20, 2026
@sonarqubecloud

Copy link
Copy Markdown

@andife andife merged commit 46dbe73 into onnx:main Mar 20, 2026
6 of 8 checks passed
@strimo378

Copy link
Copy Markdown
Collaborator Author

@andife I haven’t tested the impact on historical data. I only verified that the JSON output of the individual backends is correct, and a field for skipped tests was already present.

More generally, what is the value of keeping historical data if it is inaccurate?

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