fix failed onnx tests
Context
This MR is intended to fix the failed onnx tests encountered in issue aidge_onnx#50 and issue aidge_onnx#46.
This includes the following problems:
- And/Equal: A mistake was made, the operator Equal was named And.
- AvgPooling: unsupported attrs
dilations
andceil_mode
- MaxPooling: unsupported attr
dilations
Detailed major modifications
The following modifications were made:
- Add Equal operator
- Fix And operator
- Add dilations attr to Maxpooling
- Add
dilations
andceil_mode
to AvgPooling
TODO
-
Add Equal Op -
Fix And Op -
AvgPooling: add dilations
,ceil_mode
attrs -
MaxPooling: add dilations
attr
Edited by Houssem ROUIS
Merge request reports
Activity
Filter activity
added Fix 🔥🔥 StatusWork in Progress TopicOperator labels
assigned to @hrouis
added 11 commits
-
727d823f...d19d9f59 - 7 commits from branch
dev
- 22eaa901 - add Equal operator
- 94a7490a - fix And operator
- e930a0eb - add dilations to maxpool
- cc6e0cc2 - add dilations and cielmode to AvgPooling
Toggle commit list-
727d823f...d19d9f59 - 7 commits from branch
added 39 commits
Toggle commit list- Resolved by Olivier BICHLER
@hrouis You marked the core MR as ready, is this ready as well?
added 1 commit
- 18e4d1a8 - separate fwdDims tests section from fwd section
changed milestone to %aidge v0.6.0
added 8 commits
-
8a669993 - 1 commit from branch
dev
- d3f9986c - add Equal operator
- fc7a77e4 - fix And operator
- 926930fd - add dilations to maxpool
- 905a05f1 - add dilations and cielmode to AvgPooling
- b5930184 - handle ceil_mode in pooling kernels
- d2a98a6b - add ceil_mode tests for Avg and Max Pooling
- 36ae31ac - separate fwdDims tests section from fwd section
Toggle commit list-
8a669993 - 1 commit from branch
enabled an automatic merge when all merge checks for 36ae31ac pass
aborted the automatic merge because the source branch was updated. Learn more.
added 8 commits
- b7ae019a - add Equal operator
- c9de2f9b - fix And operator
- 969148ae - add dilations to maxpool
- 6fdef112 - add dilations and cielmode to AvgPooling
- 07507b99 - handle ceil_mode in pooling kernels
- 9f2f47ca - add ceil_mode tests for Avg and Max Pooling
- 18e4d1a8 - separate fwdDims tests section from fwd section
- 1e39f663 - remove unnecessary header in Equal tests
Toggle commit listadded 9 commits
-
8a669993 - 1 commit from branch
dev
- 97d0996a - add Equal operator
- 8701618c - fix And operator
- 0fbfb571 - add dilations to maxpool
- 96cea9f4 - add dilations and cielmode to AvgPooling
- c12d2826 - handle ceil_mode in pooling kernels
- b3ae66f7 - add ceil_mode tests for Avg and Max Pooling
- 40a34dc2 - separate fwdDims tests section from fwd section
- f3de3e10 - remove unnecessary header in Equal tests
Toggle commit list-
8a669993 - 1 commit from branch
requested review from @olivierbichler
Please register or sign in to reply