Remove onnxsim dependency from DaCeML
Open
enhancement
- Dominant language
- Python
- Stars
- 593
- Forks
- 163
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 60
Description
We currently use `onnxsim` to simplify the ONNX graph before transforming it into an SDFG. The main benefit we currently get from this is that `onnxsim` applies constant folding, which allows us to handle much simpler cases for the supported ops, such as only having constant inputs for some transformation parameters, like the axes of a reduction.
`onnxsim` is not a reliable dependency and should be replaced with a constant folding pass that simplifies the SDFG at the same level.
Contributor guide
Assessment
This issue has not been assessed yet.