Skip to content

Increase wheel size cap to 525M account for newer architectures#7116

Merged
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.10from
divyegala:increase-wheel-cap
Aug 16, 2025
Merged

Increase wheel size cap to 525M account for newer architectures#7116
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.10from
divyegala:increase-wheel-cap

Conversation

@divyegala
Copy link
Copy Markdown
Member

@divyegala divyegala commented Aug 14, 2025

Increasing for the addition of arch 121 which increases the wheel size fromm ~487M to ~507M.

@divyegala divyegala requested a review from a team as a code owner August 14, 2025 17:55
@divyegala divyegala requested a review from gforsyth August 14, 2025 17:55
@github-actions github-actions Bot added the Cython / Python Cython or Python issue label Aug 14, 2025
Comment thread python/libcuml/pyproject.toml Outdated

# detect when package size grows significantly
max_allowed_size_compressed = '500M'
max_allowed_size_compressed = '550M'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
max_allowed_size_compressed = '550M'
max_allowed_size_compressed = '525M'

I'll approve this as is, but what do you think about a 525M cap?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, can do actually. A stricter cap will help us triage situations like today.

@divyegala divyegala changed the title Increase wheel size cap to 550M account for newer architectures Increase wheel size cap to 525M account for newer architectures Aug 14, 2025
@divyegala divyegala added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Aug 14, 2025
@jcrist
Copy link
Copy Markdown
Member

jcrist commented Aug 16, 2025

/merge

@rapids-bot rapids-bot Bot merged commit d0acd48 into rapidsai:branch-25.10 Aug 16, 2025
196 of 201 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cython / Python Cython or Python issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants