The GitHub Actions job "tvm-bot" on tvm.git/main has succeeded.
Run started by GitHub user tlopex (triggered by tlopex).

Head commit for run:
6569cf0ac968f237ca6ad5f19216577cd5cb6230 / Neo Chien 
<[email protected]>
[Relax][ONNX] Fix CumSum axis handling: support runtime axis tensor, error on 
multi-element axis (#19467)

Hi Committers,

This PR is trying to fix issues
https://github.com/apache/tvm/issues/19437. Any suggestions would be
appreciated if you are available.

### Root Cause
The original CumSum converter always defaulted to axis=0 when the axis
input was a relax.Var (i.e., a runtime tensor), ignoring the actual
runtime value. This led to incorrect behavior and did not comply with
the ONNX specification.

### Solutions
Update CumSum._impl_v14 to:
- Check if the axis input is a Constant: require it to have exactly one
element, otherwise raise an error.
- If the axis input is a relax.Var, raise an error instead of always
defaulted to axis=0.

---------

Co-authored-by: cchung100m <[email protected]>

Report URL: https://github.com/apache/tvm/actions/runs/25212567175

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to