Skip to content
Snippets Groups Projects

Add erf operators and unit tests

Merged Matthew Newson requested to merge mnewson/aidge_export_cpp:erf into dev

Context

While trying to export the ConvNeXt onnx model to cpp, we encountered several issues. The one of the issue was the missing operator erf.

Modified files

Added erf operator :

  • erf.hpp added erf operator;
  • erf_config.jinja, erf_forward.jinja template jinja file for erf;
  • test_export.py, unit test for the erf operator;
  • operator.py, register erf operator;

TODO

Added erf operator for ConvNeXt model

  • NOT DONE
  • DONE
  • TO DO

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Hello @mnewson thank you for you contribution!

    Just a few changes required and you will also need to rebase with the new changes.

    To adapt to the refactoring you just need to create erf.py in the operator folder and copy what you added in operators.py.

  • Cyril Moineau requested changes

    requested changes

  • Matthew Newson added 1 commit

    added 1 commit

    • 0fef7aa2 - Debug includes deleted and omp modification

    Compare with previous version

  • Matthew Newson resolved all threads

    resolved all threads

  • Matthew Newson added 98 commits

    added 98 commits

    Compare with previous version

  • Looks like you still have an issue needing to rebase.

  • Matthew Newson added 1 commit

    added 1 commit

    • 59b3c8f8 - Test export now clean successfull tests.

    Compare with previous version

  • Matthew Newson added 1 commit

    added 1 commit

    Compare with previous version

  • Matthew Newson added 1 commit

    added 1 commit

    • a75766f0 - Delete operator.py for merge because of conflit

    Compare with previous version

  • Matthew Newson added 99 commits

    added 99 commits

    • a75766f0...9f86629b - 98 commits from branch eclipse/aidge:dev
    • 4dfc0210 - Merge branch 'main' of https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp into erf

    Compare with previous version

  • Matthew Newson added 1 commit

    added 1 commit

    Compare with previous version

  • Matthew Newson added 2 commits

    added 2 commits

    • 52348ca9 - 1 commit from branch eclipse/aidge:dev
    • 6314b2c4 - Merge branch 'dev' of https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp into erf

    Compare with previous version

  • Matthew Newson added 1 commit

    added 1 commit

    • da35647a - Add unit test for erf and include math.h

    Compare with previous version

  • Cyril Moineau enabled an automatic merge when all merge checks for da35647a pass

    enabled an automatic merge when all merge checks for da35647a pass

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading