Skip to content

Release v0.3.0

Grégoire Kubler requested to merge dev into main

aidge_onnx v0.3.0

contributors

Thanks to all of you !

🥡 About the operators

And

Node import and export !73 (merged)solved by Houssem Rouis

Argmax

Node import and export !73 (merged) solved by Houssem Rouis

AveragePool

Node export !68 (merged) solved by Grégoire Kubler

ConstantOfShape

Node import & export !70 (merged) solved by Grégoire Kubler

GlobalAveragePooling

Node import & export !30 (merged) solved by grégoire Kubler

Identity

Added node export !68 (merged) solved by Grégoire Kubler

MatMul

Node import !49 (merged) solved by Christophe guillon

PaddedAveragePooling

Added node export !68 (merged) solved by Grégoire Kubler

ReduceMean

The operator is not templated anymore, hence reduce mean can be used on N dimensions. !42 (merged) solved by Houssem ROUIS

ReduceSum

Node import and export !73 (merged) solved by Houssem Rouis

Reshape
  • Added support for allow_zero attribute !42 (merged) solved by Houssem ROUIS
  • Support both cases where the shape is either an input or an attribute !38 (merged) solved by Houssem ROUIS
Shape

Node import & export !46 (merged) solved by Houssem Rouis

Split

Node import & export !46 (merged) solved by Houssem Rouis

Squeeze

Node import & export !69 (merged) solved by Grégoire Kubler

Unsqueeze

Node import & export !69 (merged) solved by Grégoire Kubler

🧰 Features

🦖 MAJOR

Rework of all operator import functions

NO API CHANGES All operator imports have been reworked to

  1. ensure all opset are supported for currently supported operators.
  2. Homogeinize & cleanup import functions Related MR : !60 (merged) solved by Lucas Lopez
Build process

NO API CHANGES

  • !41 (merged) solved by Grégoire Kubler
    • Removal of setup.py in favor of a pyproject.toml
    • CI support for automatic packaging and deploy
    • CI files are now common to every project to ease integration of more static analysis tools
  • Support for editable installs !64 (merged) solved by Christophe Guillon

🦎 MINOR

Better error message when importing a producer with no implementation

Related MR : [Export] Producer with no implementation throw error solved by Cyril Moineau

Node imported from onnx are no longer named after the onnx node name

Now nodes are names using the CreateUniqueName function Since commit aidge_core@c916dde6

Completed tests for converter registers

!68 (diffs) Solved by Grégoire Kubler

🔥🐛 Fixes

Export

  • Allow to export graphview with no input node : #24 (closed) solved by Olivier BICHLER
  • If a layer is marked as output and outputs also to another layer, the export would generate the output twice #23 (closed) solved by Cyril Moineau
  • fix wrong return type for aidge to onnx node converters !67 (merged) solved by Grégoire Kubler
  • Fix issues with onnx export: unable to export a GraphView with no inputs !51 (merged) solved by Olivier BICHLER
  • Checking outputs_dims is not None before trying to access values.(!45 (merged)) solved by Cyril Moineau

Import

  • Onnx model opsets are now correctly retrieved: Previously, its the ir_version that was retrieved, this fixes goes with the rework of the operators imports !60 (f8d9f415) solved by Grégoire Kubler
  • Split output number is no longer decided regarding the attribute value but rather the number of outputs the node has: !56 (merged) solved by Houssem Rouis
  • Aidge Graph outputs are now ordered like in onnx graph !66 (merged) solved by Olivier BICHLER

Nodes

FC

Node export fail if it is the first layer due to a lack of checks that the input exist when checking if we need to insert a Flaten node !40 (merged) solved by Cyril Moineau

Gather

Set default axis to 0 as it is for onnx !42 (merged) solved by Houssem ROUIS

Edited by Grégoire Kubler

Merge request reports