-
Notifications
You must be signed in to change notification settings - Fork 6
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
Inaccuracy in documentation regarding installation through pip #83
Comments
Overall, this is very strange, because as far as I can see, your CMakeLists by default build should happen in such a way that the perf-analyzer is added to the python client wheel |
I'd like to share some feedback and seek clarification regarding the process of building from source as documented, too. It appears the documentation might not clearly specify the necessary details for building from sources. I successfully built
From my understanding, the OpenSSL options could be optional, whereas the Triton-specific options are crucial to ensure the build does not default to the main branch. Could you confirm if my understanding is correct? Additionally, I encountered an issue with a missing If we could also discuss the pip issue and any other outstanding questions, I would be happy to contribute a small PR to enhance the documentation based on our discussion. |
@azsh1725 can you share more details regarding the error you encountered when you pip installed tritonclient? What environment are you running in? One of our release images? Custom location? I installed I believe the OpenSSL libs are required to build the client. If you remove that option from the build does it fail? Regarding the Triton-specific options, you are correct--they will default to main unless specifically overwritten. |
Hello!
I’ve noticed that the information in the install section for pip is outdated. Specifically, after running
i got error whe try to run
perf_analyzer
command. As I understand it, this is related to the latest refactoring that the triton team did to move perf_analyzer to a separate repository.This might confuse users who are working with the project.
The text was updated successfully, but these errors were encountered: