feat/script_compare_layers_aidge_onnx
Context
Since I had problems isolating the issue in a network I wrote with the help of @hrouis a little script that compares layer per layer, the output of an onnx network and its aidge equivalent.
This script is purposed to be called as a function or as a standalone tool so it can be easily be integrated to tests later on.
It misses some small features such as support for nodes with multiple different output and creation of inputs for network with multiple inputs. other than that I feelit doesn't need much more. If you see any feature that needs to be added feel free to tell me.
Merge request reports
Activity
mentioned in issue aidge_backend_cpu#7
added 1 commit
- 59fe9b4b - chore : code cleanup + function arg type hint
-
Good util script !
I was a little sceptical about the use of ``find_node_with_name```as I thought we could use dictionnary, but change is not trivial and the code lgtm. Let's not over complicate things :)
Also I think we should move
onnx2aidge_nameinutils.pyand update utils script to use this.This way if we identify a new caracter we do not support, we will not have to modify two functions.
added 6 commits
- 2ad5f20a - feat : adding support for GlobalAveragePooling operator
- 0284e294 - Merge branch 'memorize' into feat/operator_globalAveragePooling
- 81a19a97 - feat : support for GlobalAveragePooling
- ebd439bb - Merge remote-tracking branch 'EclipseRepo/dev' into feat/operator_globalAveragePooling
- ea0508e4 - Merge branch 'feat/operator_globalAveragePooling' into feat/script_compare_layers_aidge_onnx
- 26361cbe - chore : moved onnx_to_aidge_name to utils.py + formatting
Toggle commit list
I resolved thread but merged with !30 (merged) for testing purpose. hence, this MR must be done before merging here.
added 1 commit
- 0e98557f - chore : refactored onnx_to_aidge_name to replace the "replace" function
added 1 commit
- 168adfa7 - chore : refatored function name and added docstring
added 3 commits
- 39b6b91f - Merge branch 'dev' of https://gitlab.eclipse.org/eclipse/aidge/aidge_onnx into...
- 3f33a0d1 - Merge branch 'feat/operator_globalAveragePooling' into feat/script_compare_layers_aidge_onnx
- dbdbc860 - chore : renamed function
mentioned in issue #11
added 12 commits
-
dbdbc860...81c00c8a - 2 commits from branch
eclipse/aidge:dev - c1f5a2f0 - fix : new signature for node importer
- 327f7352 - fix : updated arguments for generatescheduling
- be434b21 - chore : added globalAveragePooling to test register import
- b816863f - fix: globalaveragepooling export
- 5dffde31 - fix : test
- a38c766e - Merge branch 'feat/operator_globalAveragePooling' into feat/script_compare_layers_aidge_onnx
- 5586cdfc - Merge branch 'dev' of https://gitlab.eclipse.org/eclipse/aidge/aidge_onnx into...
- 254a0f6a - trigger CI
- df91d8c1 - Merge branch 'feat/operator_globalAveragePooling' into feat/script_compare_layers_aidge_onnx
- 241c108a - fix : inverted conditions on onnx2aidge_name
Toggle commit list-
dbdbc860...81c00c8a - 2 commits from branch
-
@cmoineau or @pineapple MR done, ready for merge after !30 (merged) is done.
assigned to @gregkub
requested review from @cmoineau
-
Every thread are closed, I validate and run the MR
added 11 commits
-
4d5527ba - 1 commit from branch
eclipse/aidge:dev - 8b7eb8b0 - feat : script for layer comparison created
- aec5fa0c - feat : added doc, some type hint and repurposed the script to be able to call...
- 3d8ccac3 - chore : code cleanup + function arg type hint
- eb62057f - feat : new "save graph" functionnality
- 7b59eeb3 - chore : moved onnx_to_aidge_name to utils.py + formatting
- 1d969af5 - chore : changed success message
- dfdabc66 - chore : refactored onnx_to_aidge_name to replace the "replace" function
- 4ec6826d - chore : refatored function name and added docstring
- e6fdc573 - chore : renamed function
- f54ee318 - fix : inverted conditions on onnx2aidge_name
Toggle commit list-
4d5527ba - 1 commit from branch
enabled an automatic merge when the pipeline for f54ee318 succeeds
mentioned in commit b0f733f0