Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Switched API backend to ONNX #224

Merged
merged 5 commits into from
Jul 14, 2022
Merged

feat: Switched API backend to ONNX #224

merged 5 commits into from
Jul 14, 2022

Conversation

frgfm
Copy link
Owner

@frgfm frgfm commented Jul 14, 2022

This PR introduces the following modifications:

  • removes pytorch and uses ONNX as DL framework for inference in the API
  • updates dependencies and functions accordingly

@frgfm frgfm added type: improvement New feature or request ext: api labels Jul 14, 2022
@frgfm frgfm added this to the 0.3.0 milestone Jul 14, 2022
@frgfm frgfm self-assigned this Jul 14, 2022
@codecov
Copy link

codecov bot commented Jul 14, 2022

Codecov Report

Merging #224 (3278c15) into main (48d74b0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #224   +/-   ##
=======================================
  Coverage   93.50%   93.50%           
=======================================
  Files          56       56           
  Lines        3127     3127           
=======================================
  Hits         2924     2924           
  Misses        203      203           

@frgfm frgfm merged commit 89e8aee into main Jul 14, 2022
@frgfm frgfm deleted the api-onnx branch July 14, 2022 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ext: api type: improvement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant