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

Add quick start documentation page #261

Merged
merged 5 commits into from
Mar 15, 2023
Merged

Conversation

matthewkotila
Copy link
Contributor

No description provided.

Copy link
Member

@Tabrizian Tabrizian left a comment

Choose a reason for hiding this comment

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

Instead of having a new page, do you think we can embed this in the Perf Analyzer readme? https://github.com/triton-inference-server/client/blob/main/src/c++/perf_analyzer/README.md#performance-analyzer

src/c++/perf_analyzer/docs/quick_start.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/quick_start.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/quick_start.md Outdated Show resolved Hide resolved
src/c++/perf_analyzer/docs/quick_start.md Outdated Show resolved Hide resolved
@matthewkotila
Copy link
Contributor Author

@Tabrizian: Instead of having a new page, do you think we can embed this in the Perf Analyzer readme? https://github.com/triton-inference-server/client/blob/main/src/c++/perf_analyzer/README.md#performance-analyzer

Moved to main PA readme 👍

@matthewkotila matthewkotila requested a review from Tabrizian March 14, 2023 00:23
@matthewkotila matthewkotila changed the base branch from main to pa-docs March 14, 2023 00:25
@matthewkotila matthewkotila force-pushed the matthewkotila-quick-start branch from 79a5a24 to 44e4272 Compare March 14, 2023 19:06
@matthewkotila matthewkotila requested a review from tgerdesnv March 14, 2023 19:07
@matthewkotila matthewkotila requested a review from tgerdesnv March 14, 2023 22:09
@matthewkotila matthewkotila changed the base branch from pa-docs to main March 15, 2023 02:05
@matthewkotila matthewkotila changed the base branch from main to pa-docs March 15, 2023 02:05
@matthewkotila matthewkotila merged commit e8f655d into pa-docs Mar 15, 2023
@matthewkotila matthewkotila deleted the matthewkotila-quick-start branch March 15, 2023 16:35
matthewkotila added a commit that referenced this pull request Mar 15, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Mar 16, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Mar 21, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Mar 22, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Mar 22, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Mar 22, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Apr 12, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Apr 13, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Apr 18, 2023
* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments
matthewkotila added a commit that referenced this pull request Apr 20, 2023
* Add quick start documentation page (#261)

* Add quick start documentation page

* Addressed comments

* Addressed comments

* Fixed typo

* Addressed comments

* Add installation documentation page (#266)

* Add installation documentation page

* Add quick start to docs readme

* Add CLI documentation page (#271)

* Add CLI documentation page

* Addressed comments

* Addressed comments

* Clean up

* Clean up

* Rewrite top-level readme (#273)

* Rewrite top-level readme

* Addressed comments

* Add input data, measurements, benchmarking documentation (#274)

* Add data guide documentation

* Add measurement doc

* Add benchmarking doc

* Add more to measurements and metrics doc and new link in readme

* Fix some comments and add more to metrics

* Move data_guide to input_data

* Adjusted header size

* Update all links

* Add back communication protocol docs, fix links, add bls composing models option (#287)

* Fix various things in docs (#288)

* Fix various things in docs

* Addressed comments

* Proof-read all docs (#290)

* Proof-read all docs

* Addressed comments

* Fix docs

---------

Co-authored-by: Matthew Kotila <matthew.r.kotila@gmail.com>
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.

3 participants