search

oramasearch / onnx-go

onnx-go gives the ability to import a pre-trained neural network within Go without being linked to a framework or library.
https://blog.owulveryck.info/2019/04/03/from-a-project-to-a-product-the-state-of-onnx-go.html
MIT License
717 stars 71 forks source link

Implement operator Clip for backend Gorgonia/Gorgonnx #173

Open mauri870 opened 4 years ago

mauri870 commented 4 years ago

Why is this operator needed?

Received this error when trying to run a MobilenetV2 model:

onnx: operator Clip not implemented ()

Implementation

Link to existing material on the backend

N/A

Expected problems?

N/A

Tests

go test -run=ONNX/TestOperator

owulveryck commented 4 years ago

Thanks for this issue.

I’ll try to have a look at it this month if I find some free time.

Le lun. 6 janv. 2020 à 12:32, Mauri de Souza Nunes notifications@github.com a écrit :

Why is this operator needed?

Received this error when trying to run a MobilenetV2 model:

onnx: operator Clip not implemented ()

Implementation

Link to existing material on the backend

N/A Expected problems?

N/A Tests

go test -run=ONNX/TestOperator

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/owulveryck/onnx-go/issues/173?email_source=notifications&email_token=ACYL7KMP34V6J7IYESATGTLQ4MJFVA5CNFSM4KDDFEOKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IEGODLQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACYL7KNPZ4TSD2J35NTH443Q4MJFVANCNFSM4KDDFEOA .

mauri870 commented 4 years ago

@owulveryck Any news on this?

owulveryck commented 4 years ago

Hi, Sorry, I lack the time. I first need to do a couple of fixes to #176 to start on a sanitized code-base. I will have a look right after that, but TBH I cannot tell you when. Meanwhile, I can give you support if you want to implement it and submit a PR if there is an emergency or you.