spcl / spcl/dace

Remove onnxsim dependency from DaCeML

Open
#2,242 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.