Conversation
- Expand deployment section - Minor wording updates
Will move to dockerfiles readme page
Should this also be applied to all other instructions for all docker run instructions on that page? |
yes, applies to all EPs. docker build and docker run does not require sudo. |
|
|
||
| ## Cross Platform | ||
| [API documentation and package installation](#installation) | ||
| ONNX Runtime is currently available for Linux, Windows, and Mac with Python, C#, C++, and C APIs. Please see [API documentation and package installation](#installation). |
There was a problem hiding this comment.
May consider adding Android here.
There was a problem hiding this comment.
we can add it once we've confirmed NN API readiness.
docs/ONNX_Runtime_Perf_Tuning.md
Outdated
| @@ -70,30 +70,32 @@ sess_options.intra_op_num_threads=2 | |||
| sess_options.enable_sequential_execution=True | |||
There was a problem hiding this comment.
sess_options.enable_sequential_execution=True
Pranav is changing the API. When his has done the work, we should review this part and refresh it again.
There was a problem hiding this comment.
CC @pranavsharma please note and include this update as part of your PR
## Describe your changes Refactor docker system: 1. If user specific docker system as host, the whole workflow will run in the docker contianer. 2. Remove pass run and evaluation run support for docker system. 3. No need to add workflow_host for docker system. 4. Remove managed docker system support. User needs to provide his own Dockerfile. 5. Remove unused docker integration test. 6. Update unit tests and docs. ## Checklist before requesting a review - [ ] Add unit tests for this change. - [ ] Make sure all tests can pass. - [ ] Update documents if necessary. - [ ] Lint and apply fixes to your code by running `lintrunner -a` - [ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes. - [ ] Is this PR including examples changes? If yes, please remember to update [example documentation](https://github.com/microsoft/Olive/blob/main/docs/source/examples.md) in a follow-up PR. ## (Optional) Issue link
5 participants
Update and clean up lots of the documentation.