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

docs(ml): clarify cuda and openvino requirements #9072

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

mertalev
Copy link
Contributor

Description

Specifies that OpenVINO should only be used with discrete GPUs since this is the current state of things. Also adds info about the minimum NVIDIA driver version.

@mertalev mertalev added the documentation Improvements or additions to documentation label Apr 25, 2024
@alextran1502 alextran1502 merged commit a90138e into main Apr 25, 2024
23 of 24 checks passed
@alextran1502 alextran1502 deleted the docs/ml-openvino-dgpu branch April 25, 2024 04:15
- On Linux (except for WSL2), you also need to have [NVIDIA Container Runtime][nvcr] installed.

#### OpenVINO

- The server must have a discrete GPU, i.e. Iris Xe or Arc. Expect issues when attempting to use integrated graphics.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Expect issues when attempting to use integrated graphics.

What are the expected issues when using integrated graphics?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One or more ML tasks will just fail, see #8226 and #8353. There have been a few attempts to fix it, but fixing one issue has led to it breaking for others.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing me to those issues. I also notice myself that facial recognition started failing a few weeks ago when using OpenVino, when previously it worked just fine. I'm using Intel N100 which has integrated graphics. I had to disable OpenVino in order to get Immich working again.

What I don't understand from this new documentation is, do we expect these new issues to never be fixed, or is this just a temporary thing that we can't use OpenVino anymore?

If it is permanently broken, why does it no longer work? Did the model get too big or complex for the N100 integrated graphics? If so, what's your minimum specification?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We updated OpenVINO and onnxruntime-openvino a month ago. That fixed two issues relating to external data and dynamic input shapes, but seems to have a major regression in iGPU compatibility.

As for fixing it, this is up to upstream in OpenVINO and/or ONNX Runtime. The prerequisites right now are just because of the current behavior in these libraries. I opened an issue about it and imagine the issue will be fixed at some point, but I don't know when.

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

Successfully merging this pull request may close these issues.

None yet

3 participants