Mr-Neutr0n commented on PR #18774: URL: https://github.com/apache/tvm/pull/18774#issuecomment-4605680733
Two notes: 1. Pushed `c0061069` — small refactor reusing the `axis` local in `_impl_v1`/`_impl_v13` per @gemini-code-assist's suggestion. Pure cleanup, no behavior change. 2. On the `cpu/pr-head` build error — I have to be honest: I can't run TVM's C++ build locally to debug it (the full build is multi-hour on a workstation and would burn the rest of the day). The error log itself isn't visible from the GitHub side; `ci.tlcpack.ai` returns a redirect without the failing test name in the API response. The Python change itself is straightforward (one helper function, no C++ surface), so the build failure is most likely a stale build cache or an unrelated flake — but I don't want to guess and burn another CI cycle. @locnd182644 — you said back in February that "the 'uneven splits' issue has been solved in PR #17757 and apply at TVM Version: 0.20.0. You can check again." The original issue (#18751) is still open, though, so I want to confirm: is #17757 actually sufficient to close #18751, in which case this PR is redundant and I'd rather close it cleanly, or is the ONNX frontend path genuinely a separate gap? Happy either way — just let me know which. -- 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]
