Adding MatMul operator
Context
While trying to export the ConvNeXt onnx model to cpp, we encountered several issues. we needed to add broadcasting support for the matmul operator.
-
matmul.hpp
add broadcasting for matmul; -
matmul_config.jinja
, andmatmul_forward.jinja
to include for matmul operator;operator.py
, register matmul operator; -
test_export.cpp
, add unit tests;
Detailed major modifications
Add matmul broadcasting and unit tests.
TODO
Added matmul broadcasting for exporting ConvNeXt model with aidge_export_cpp;
-
NOT DONE -
DONE -
TO DO
Merge request reports
Activity
assigned to @mnewson
requested review from @cmoineau
added StatusReview Ready label
added Feature 🚀 label
changed milestone to %aidge v0.7.0
- Resolved by Matthew Newson
- Resolved by Matthew Newson
- Resolved by Matthew Newson
- Resolved by Matthew Newson
added StatusChanges required label and removed StatusReview Ready label
added 102 commits
- 26f6458b...f95277a8 - 92 earlier commits
- 9c729bff - Improve example script test to show stdout in failing cases.
- af97383b - Fixed missing operators
- 6f41e44a - Changed skip to expectedFailure
- f796654c - Fixed Transpose
- a3fd60a4 - Fixed reshape
- ead74936 - Update 0.3.0 -> 0.3.1
- a227c91d - [Refactor] New operators format (operators.py -> [operator_name].py) for the following operators :
- 8f7acfae - Add broadcasting for matmul and unit tests
- 6350d174 - adding previous unit tests
- 58767c65 - Add activation and rescaling
Toggle commit listadded 14 commits
- 58767c65...c35b6eea - 4 earlier commits
- c1cc47fa - Debug includes deleted and omp modification
- 36f54bae - Add erf operators and unit tests
- 49561d33 - Test export now clean successfull tests.
- 796d9288 - Add broadcasting for matmul and unit tests
- 8f2fbcad - adding previous unit tests
- 71fa08f0 - [Chore] Temporarily comment the save_outputs generated code
- d6c6ce37 - [Feat] Add int8 support
- b36348b8 - [Refactor] Split rescaling.hpp and activation.hpp into two types of files :
- b6679cc9 - Add broadcasting for matmul and unit tests
- 9c4abace - adding previous unit tests
Toggle commit listadded 100 commits
-
9c4abace...52348ca9 - 99 commits from branch
eclipse/aidge:dev
- 3566c343 - Merge branch 'dev' of https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp into MatMul
-
9c4abace...52348ca9 - 99 commits from branch
@mnewson Please rebase.
added StatusReview Ready label and removed StatusChanges required label
mentioned in merge request !33 (closed)
added 25 commits
-
8b9a9a1d...520348e5 - 23 commits from branch
eclipse/aidge:dev
- 9263f2ef - Merge branch 'dev' of https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp into MatMul
- 9ffbb22f - Merge branch 'MatMul' of gitlab.eclipse.org:mnewson/aidge_export_cpp into MatMul
-
8b9a9a1d...520348e5 - 23 commits from branch
added 3 commits
-
9ffbb22f...8f876b0b - 2 commits from branch
eclipse/aidge:dev
- 6efdddc2 - Merge branch 'dev' of https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp into MatMul
-
9ffbb22f...8f876b0b - 2 commits from branch
added 13 commits
-
8d6911af...5ab76106 - 12 commits from branch
eclipse/aidge:dev
- 03009d35 - Merge branch 'dev' into MatMul
-
8d6911af...5ab76106 - 12 commits from branch
enabled an automatic merge when all merge checks for 03009d35 pass