Draft: Fix import tests
Context
This MR is dedicated to enhance some operators imports mentioned in issue aidge_onnx#46.
The operators that will be enhanced are the following:
-
gemm
: * missingalpha
andbeta
attributes.
* missingtransA
andtransB
attributes.
* cannot support bias of shape [1, outChannels]. -
Sum
: can no longer useAdd
operator because the latter can no longer support more than 2 inputs.
Modified files
-
FCImpl.hpp
,FCImpl.cpp
,FCImpl_kernels.hpp
andTest_FCImpl.cpp
, add the attrsalpha
,beta
,transA
andtransB
and support bias of shape [1, outChannels]; -
SumImpl.hpp
,SumImpl.cpp
,SumImpl_kernels.hpp
andTest_SumImpl.cpp
, addSum
operator;
TODO
-
Add alpha
andbeta
attr toFC
operator -
Add transA
andtransB
attr toFC
oprator -
Support bias of shape [1, outChannels] in FC
oprator -
Add Sum
operator
Edited by Houssem ROUIS
Merge request reports
Activity
Filter activity
changed milestone to %aidge v0.6.0
added Enhancement ⭐ StatusWork in Progress TopicOperator labels
assigned to @hrouis
added 50 commits
-
78610889...142e6e34 - 43 commits from branch
dev
- 01d1f85d - add alpha and beta attr to FC impl
- 67c16329 - support 2D Bias for FC operator
- cfed4c24 - add transA and transB for FC
- cb246d41 - support only bias of size outChannels for FC
- 97e3ea19 - update FC factory call
- da613b4e - add hints to support batched biad in FC
- a3bd6c57 - add Sum operator
Toggle commit list-
78610889...142e6e34 - 43 commits from branch
added 1 commit
- 52f796d9 - remove transA and transB attr from FC to be handled in the metaop TransposeFC
added 33 commits
-
52f796d9...9765d6e8 - 25 commits from branch
dev
- f4a5d905 - add alpha and beta attr to FC impl
- a52c842f - support 2D Bias for FC operator
- a59730f1 - add transA and transB for FC
- ef53239c - support only bias of size outChannels for FC
- 74899817 - update FC factory call
- 0eb2f2b4 - add hints to support batched biad in FC
- 5c9c92b7 - add Sum operator
- 32c64230 - remove transA and transB attr from FC to be handled in the metaop TransposeFC
Toggle commit list-
52f796d9...9765d6e8 - 25 commits from branch
changed milestone to %aidge v0.7.0
Please register or sign in to reply