Skip to content

Update docs to remove mentions of GPU support in ApplyOnnxModel #4872

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.

Sign up for GitHub

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

Jump to bottom
Closed
gvashishtha opened this issue Feb 21, 2020 · 1 comment · Fixed by #4966
Closed

Update docs to remove mentions of GPU support in ApplyOnnxModel #4872

gvashishtha opened this issue Feb 21, 2020 · 1 comment · Fixed by #4966
Assignees
@natke
Labels
documentation Related to documentation of ML.NET P3 Doc bugs, questions, minor issues, etc.

Comments

@gvashishtha
Copy link
Contributor

When Onnxruntime released 1.0, they changed the API and broke the interface from 0.5 so that now it is not possible to ever exercise the GPU code path unless someone compiles ML.NET privately and links it to Microsoft.ML.OnnxRuntime.Gpu specifically.

Related to #4478 and #4656
@antoniovs1029 antoniovs1029 added documentation Related to documentation of ML.NET P3 Doc bugs, questions, minor issues, etc. labels Feb 28, 2020
@antoniovs1029
Copy link
Member

PR #4919 has reenabled ONNX support for GPU. Now to use ApplyOnnxModel, users should install either Onnxruntime (for CPU) or Onnxruntime.Gpu.

@antoniovs1029 antoniovs1029 mentioned this issue Mar 24, 2020
Merged
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@natke natke

Labels
documentation Related to documentation of ML.NET P3 Doc bugs, questions, minor issues, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Updated OnnxScoringEstimator's documentation antoniovs1029/machinelearning
3 participants
@natke @gvashishtha @antoniovs1029