-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Update readme for OVEP #12122
Update readme for OVEP #12122
Conversation
e4e9e8a
to
05aefef
Compare
requirements-training.txt
Outdated
@@ -4,6 +4,6 @@ h5py | |||
numpy >= 1.16.6 | |||
onnx | |||
packaging | |||
protobuf |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove this change. we're taking care of it with #12132
the problem with pinning to 3.17.0 is anything between 3.12.2 and 3.20.1 is valid (from onnx perspective)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revoked the change @jywu-msft . Kindly review and merge the changes.
05aefef
to
1bc0b33
Compare
/azp run Linux CPU CI Pipeline, Linux CPU Minimal Build E2E CI Pipeline, Linux GPU CI Pipeline, Linux GPU TensorRT CI Pipeline, Linux Nuphar CI Pipeline, MacOS CI Pipeline, ONNX Runtime Web CI Pipeline, Windows CPU CI Pipeline, Windows GPU CI Pipeline |
Azure Pipelines successfully started running 9 pipeline(s). |
/azp run Windows GPU TensorRT CI Pipeline, onnxruntime-binary-size-checks-ci-pipeline, onnxruntime-python-checks-ci-pipeline, orttraining-linux-ci-pipeline, orttraining-linux-gpu-ci-pipeline, orttraining-ortmodule-distributed, Linux OpenVINO CI Pipeline |
Azure Pipelines successfully started running 7 pipeline(s). |
Update protobuf==3.17.0 in training requirement for installation.
Update pypi Readme for OVEP