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
-
FC.hpp
,FC.cpp
andpybind_FC.cpp
, add the attrsalpha
,beta
,transA
andtransB
and support bias of shape [1, outChannels]; -
Sum.hpp
,Sum.cpp
andpybind_Sum.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 78 commits
-
ae2ea361...0e89cde0 - 71 commits from branch
dev
- a56bbbbd - add alpha and beta attr to FC
- 08199590 - support 2D bias in FC
- bff43f0c - add transA and transB attr for FC
- aab35aab - fix python binding of FC_Op
- a5382e31 - support only bias of size outChannels for factory func limitations
- a7c6ec20 - reorder FC factory function arguments
- 4cf1845b - add Sum operator
Toggle commit list-
ae2ea361...0e89cde0 - 71 commits from branch
added 1 commit
- 47001c91 - add TransposeFC MetaOp and remove transA and transB attr from FC
added 39 commits
-
47001c91...2afc296e - 31 commits from branch
dev
- a50fa266 - add alpha and beta attr to FC
- eaa8ff98 - support 2D bias in FC
- e95ac474 - add transA and transB attr for FC
- f2de6fbb - fix python binding of FC_Op
- 19e303d5 - support only bias of size outChannels for factory func limitations
- b9016985 - reorder FC factory function arguments
- 5c13d82d - add Sum operator
- 312cdbf7 - add TransposeFC MetaOp and remove transA and transB attr from FC
Toggle commit list-
47001c91...2afc296e - 31 commits from branch
Please register or sign in to reply