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

Provide 24.11 version bumps #413

Merged
merged 2 commits into from
Nov 6, 2024
Merged

Provide 24.11 version bumps #413

merged 2 commits into from
Nov 6, 2024

Conversation

hcho3
Copy link
Collaborator

@hcho3 hcho3 commented Nov 5, 2024

Using custom branch (feature/fil-backend-r24.11) for cuML and Treelite to support latest C++ compilers.

@mc-nv mc-nv requested a review from wphicks November 5, 2024 22:53
git \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/* \
&& conda run --no-capture-output -n triton_test pip install tritonclient[all]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

tritonclient[all] has dependency constraint geventhttpclient<=2.0.2, but there isn't a binary wheel for geventhttpclient 2.0.2 that targets Python 3.12 (version used in the base Triton container). So we need build-essential to be able to install geventhttpclient from the source distribution.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oof, that's unfortunate, but no way around it. Seems fine by me.

git \
&& apt-get clean \
&& rm -rf /var/lib/apt/lists/* \
&& conda run --no-capture-output -n triton_test pip install tritonclient[all]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Oof, that's unfortunate, but no way around it. Seems fine by me.

@hcho3 hcho3 merged commit 5533dfc into main Nov 6, 2024
1 check passed
@hcho3 hcho3 deleted the bump_24.11 branch November 6, 2024 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants