Adding broadcasting for elem wise operator
Context
While trying to export the ConvNeXt onnx model to cpp, we encountered several issues. We needed to add broadcasting support for operators like add, sub, mul, and div.
-
elemwise.hpp
, modified for broadcasting and added div operator; -
elemwise_config.hpp
andelemwise_fowrad.hpp
modified for broadcasting; -
test_export.py
, added new unit tests for operator div and broadcasting working for (add, mul, sub ) -
ElemeWise.py
, registered div operator
Detailed major modifications
- Implemented missing operators (div) and support for broadcasting for elemwise operators in C++.
- Added unit tests.
TODO
Fonctionnal unit test for Div.
-
NOT DONE -
DONE -
TO DO
Merge request reports
Activity
assigned to @mnewson
requested review from @cmoineau
changed milestone to %aidge v0.7.0
added Feature 🚀 StatusReview Ready labels
added StatusWork in Progress label and removed StatusReview Ready label
mentioned in issue #31 (closed)
added 99 commits
-
e8cf966d...9f86629b - 98 commits from branch
eclipse/aidge:dev
- 884b8541 - Merge branch 'main' of...
-
e8cf966d...9f86629b - 98 commits from branch
added 2 commits
added StatusReview Ready label and removed StatusWork in Progress label
@mnewson Please rebase.
mentioned in merge request !33 (closed)
added 25 commits
-
fbe6e4b0...520348e5 - 23 commits from branch
eclipse/aidge:dev
- e456e323 - Merge branch 'dev' of...
- fbb2c990 - Merge branch 'elemwise_operators' of...
-
fbe6e4b0...520348e5 - 23 commits from branch
@mnewson I am ready to merge, you just need to rebase once more!
added 3 commits
-
fbb2c990...8f876b0b - 2 commits from branch
eclipse/aidge:dev
- 2f93467f - Merge branch 'dev' of...
-
fbb2c990...8f876b0b - 2 commits from branch
enabled an automatic merge when all merge checks for 2f93467f pass
Requires aidge_core!457 (merged) to pass.
enabled an automatic merge when all merge checks for 5ab76106 pass