Add unit test

• Add a method to automatically test exports (see #17 (closed))
• Add a test for the export in generic C++ of FC with a flatten input
• Add a test for the export in generic C++ of FC with a 3D input.
• Add a test for a Conv (note: test failing so I skipped it as the objective of this MR is only to introduce unit tests)

assigned to @cmoineau

requested review from @axelfarr

marked this merge request as draft

requested review from @pineapple

added 1 commit

• a84ee5e3 - Add Conv unit test.

Compare with previous version

@pineapple FYI

mentioned in issue #12 (closed)

FYI @gallasko. @cmoineau Gallas is also looking at improving unit tests for aidge_export_cpp,

Ok! this MR is in draft and I am currently working on exporting quantized models with ONNX. If you have any updates, feel free to create a MR. I will update you when i start to work back on this.

Great that you're working on quantized model export. @gallasko will probably push a MR at some point, indeed.

added 20 commits

• a84ee5e3...58c0e326 - 18 commits from branch dev
• 97d1e484 - Merge branch 'dev' into AddUnitTest
• 47ff919e - Fix test_export.py

Compare with previous version

added 1 commit

• 0698a5f0 - Update test_export with unit_test_export as a method instead of an external function.

Compare with previous version

mentioned in merge request !33 (closed)

added 1 commit

• 05387098 - Add docstring + small clean.

Compare with previous version

marked this merge request as ready

added StatusReview Ready label

changed the description

changed milestone to %aidge v0.6.0

added CI 👷 TopicExport Unit-test 🧪 labels

@axelfarr @gallasko @cguillon @mick94 this MR is quite old (4months ago...). I made this MR when I realized that FC and Conv did not gave the right reults. The problem is that the fix I created for FC and Conv are outdated (and i did not fully fixed Conv ...). So this MR contains a lot of changes I didn't remember and that may break existing MR (!32 (merged), !33 (closed)).

In order to avoid this, I will close this MR and create a new one pushing only the new unit test.

**Note:** Conv has been fixed in dev but not the FC which this branch contains a valid fix off, so I will skip the FC and add an issue to cherry pick the fix for FC from this branch.

closed

mentioned in merge request !34 (merged)