The GitHub Actions job "CI" on tvm-ffi.git/pyobject has failed. Run started by GitHub user cyx-6 (triggered by cyx-6).
Head commit for run: 3ac5852fc7a8d0b11d45ffe40d61a5eb52576c13 / Yaxing Cai <[email protected]> [DOCS][Python] Simplify PyObject-tying comments in cython sources Documentation-only cleanup of the comments added when the wrapper-tie landed; no code or behavior change. tvm_ffi_python_helpers.h: - Trim the top-of-file overview (states, invariants, handshake, slot install, free-threaded section) to an overview that defers implementation detail to the code, removing double-statement. - Rewrite the hand-built tp_dealloc slot block: pseudo-code for the Cython-thunk-vs-ours contrast, a routing call-flow comparison keyed on what `o` is (cdef subtype = compile-time symbol; heap subtype = subtype_dealloc runtime slot walk), and a brief note on the rejected backup-and-wrap alternative. .pxi files: - Drop the stale "pre-bump" framing now that the slot fully replaces Cython's thunk (the wrapper is never bumped on FT). - Fix a stale symbol reference in tensor.pxi (TVMFFIPyAttachPyObject -> the binding is simply not made canonical here; the real helper is TVMFFIPyRebindPyObject). - Collapse the four identical carrier-install one-liners to defer to the full explanation in object.pxi, and tighten the rvalue-ref comment. Verified on the free-threaded 3.14t build: rebuild clean, tying tests 43/43, crash oracle 5/5, full suite 2333 passed. Report URL: https://github.com/apache/tvm-ffi/actions/runs/27708728023 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
