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

Update README for 22.03 release #4119

Merged
merged 1 commit into from
Mar 29, 2022
Merged

Update README for 22.03 release #4119

merged 1 commit into from
Mar 29, 2022

Conversation

mc-nv
Copy link
Contributor

@mc-nv mc-nv commented Mar 28, 2022

No description provided.

@mc-nv mc-nv requested a review from dzier March 28, 2022 19:03
README.md Outdated
application.

## What's New in 2.20.0
- Models can now [load from a serialized model_config message](docs/protocol/extension_model_repository.md#load) with the Triton Server API.
Copy link
Collaborator

Choose a reason for hiding this comment

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

In the readme.md, lines should not be more than 80 characters. You will need to break them up. Also, you will need to add empty lines after headers and between each '-'.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

README.md Outdated
- ONNX Runtime, TensorRT, and Tensorflow backends now support server-side, multi-dimensional [ragged batching](docs/ragged_batching.md).
- [Cache miss statistics](docs/metrics.md) have been added to the Prometheus metrics.
- Trace settings can be configured with the [Triton Server Trace Protocol](docs/protocol/extension_trace.md).
- Refer to the 22.03 column of the [Frameworks Support Matrix](https://docs.nvidia.com/deeplearning/frameworks/support-matrix/index.html) for container image versions on which the 22.03 inference server container is based.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This line does not get added to the What's New section. Remove

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

README.md Outdated
*developer* documentation describes how to build and test Triton and
also how Triton can be extended with new functionality.

The Triton [Release
Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove the lines 114 to 119 here. They are only for the main branch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@mc-nv mc-nv marked this pull request as ready for review March 28, 2022 20:58
README.md Outdated
application.

## What's New in 2.20.0
- Models can now [load from a serialized model_config message](docs/protocol/extension_model_repository.md#load)
Copy link
Collaborator

Choose a reason for hiding this comment

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

You still need empty line breaks. And use * instead of -. See how the list is created in the Features section for reference on spacing.

@mc-nv mc-nv force-pushed the mchornyi-r22.03 branch 3 times, most recently from fe2b61d to a639d13 Compare March 28, 2022 22:53
@mc-nv mc-nv merged commit e8d7325 into r22.03 Mar 29, 2022
@dzier dzier deleted the mchornyi-r22.03 branch March 29, 2022 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants