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

PA Migration: Update docs #254

Merged
merged 1 commit into from
Aug 8, 2024
Merged

PA Migration: Update docs #254

merged 1 commit into from
Aug 8, 2024

Conversation

fpetrini15
Copy link
Contributor

@fpetrini15 fpetrini15 commented Aug 5, 2024

Copy link
Collaborator

@szkarpinski szkarpinski left a comment

Choose a reason for hiding this comment

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

@fpetrini15 The new link appears to be broken (https://github.com/triton-inference-server/perf_analyzer/blob/main/README.md) and https://github.com/triton-inference-server/perf_analyzer/ repo seems not to be live yet.

Please let us know when it's there and we'll be happy to merge this. Thanks!

may be found in the [`perf_analyzer` documentation](https://github.com/triton-inference-server/perf_analyzer/blob/main/README.md).
Copy link
Collaborator

Choose a reason for hiding this comment

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

@fpetrini15
Copy link
Contributor Author

Correct, the repo is still private. We are aiming to switch it to public shortly. I'll let you know when that happens!

@fpetrini15
Copy link
Contributor Author

The repo is now public. Please check again.

@fpetrini15 fpetrini15 requested a review from szkarpinski August 7, 2024 22:00
@fpetrini15
Copy link
Contributor Author

@szkarpinski You'll have to merge as well. I am not authorized.

@JanuszL JanuszL merged commit b012a9f into main Aug 8, 2024
@JanuszL JanuszL deleted the fpetrini-purge-pa-from-client branch August 8, 2024 15:37
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.

4 participants