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

upgrading to ort 2.0.0-rc.2, onnxruntime 1.17.3 #28

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

michaelbearne
Copy link

Hi

I had an issue getting the Ortex compiled in a Docker container, and then an issue running the tests on a Mac, which I thought might have been caused by downloading the Onnx runtime shared libraries.

So, I tried upgrading to the latest Rustler and Ortex. With these updates, I don't seem to need the libs copy workaround in lib/vortex/native.ex as now one lib is created. I have only tested this on Mac and Linux without Cuda.

The issue with not working on Mac was the Statically linking to CoreML bindings needing the link flag
"-C link-arg=-fapple-link-rtlib" according to the Ort docs.

The docker compile was a path issue with cargo, which I only figured out after updating the dependencies.

@gregszumel
Copy link

Hey @michaelbearne -- thanks for this PR!

I tried running mix test on my machine and had some lib-not-found issues, so maybe ort is changing the way it handles downloading libraries between rc-0 and rc-2?

I think we'll unfortunately need to hold off on merging this into main for now, at least until ort's API is more settled, or until I get more capacity to debug why I'm getting those errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants