Add erf operators and unit tests
Context
While trying to export the ConvNeXt onnx model to cpp, we encountered several issues. The one of the issue was the missing operator erf.
Modified files
Added erf operator :
-
erf.hpp
added erf operator; -
erf_config.jinja
,erf_forward.jinja
template jinja file for erf; -
test_export.py
, unit test for the erf operator; -
operator.py
, register erf operator;
TODO
Added erf operator for ConvNeXt model
-
NOT DONE -
DONE -
TO DO
Merge request reports
Activity
assigned to @mnewson
requested review from @cmoineau
added Feature 🚀 StatusReview Ready labels
- Resolved by Matthew Newson
- Resolved by Matthew Newson
Hello @mnewson thank you for you contribution!
Just a few changes required and you will also need to rebase with the new changes.
To adapt to the refactoring you just need to create
erf.py
in the operator folder and copy what you added inoperators.py
.added 98 commits
- 0fef7aa2...71503da8 - 88 earlier commits
- 36c70775 - Add aidge_quantization dependencie for tests.
- c64e5a83 - Add torch dep for tests.
- 68437da6 - Clean import in resnet18 script.
- 688291f2 - Improve example script test to show stdout in failing cases.
- 0862d719 - Fixed missing operators
- 573cf314 - Changed skip to expectedFailure
- 180fa202 - Fixed Transpose
- d39e7ef4 - Fixed reshape
- 79af4787 - Update 0.3.0 -> 0.3.1
- 78632389 - Add erf operators and unit tests
Toggle commit listadded 1 commit
- a75766f0 - Delete operator.py for merge because of conflit
added 99 commits
-
a75766f0...9f86629b - 98 commits from branch
eclipse/aidge:dev
- 4dfc0210 - Merge branch 'main' of https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp into erf
-
a75766f0...9f86629b - 98 commits from branch
enabled an automatic merge when all merge checks for da35647a pass
- Resolved by Olivier BICHLER
Why not use
std::erf()
?