Skip to content

Conversation

@brendan-priorlabs
Copy link
Contributor

Our mac builds with Python 3.9 were all failing because an indirect dep stopped publishing packages for 3.9. See https://github.com/PriorLabs/TabPFN/actions/runs/18689219131/job/53367028472?pr=539

This fixes it by pinning away for the old builds.

@brendan-priorlabs brendan-priorlabs requested a review from a team as a code owner October 22, 2025 18:43
@brendan-priorlabs brendan-priorlabs requested review from noahho and removed request for a team October 22, 2025 18:43
@brendan-priorlabs brendan-priorlabs enabled auto-merge (squash) October 22, 2025 18:43
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request aims to fix a build failure on Python 3.9 by pinning pyobjc-framework-Metal and its indirect dependency pyobjc-core. While the intention is correct, the specified version range is too broad and will not resolve the underlying issue. I have provided a critical suggestion to adjust the version pinning to correctly exclude the problematic versions and ensure the build succeeds on Python 3.9.

Copy link
Contributor

@bejaeger bejaeger left a comment

Choose a reason for hiding this comment

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

Thanks!

@brendan-priorlabs brendan-priorlabs merged commit b215940 into main Oct 22, 2025
10 checks passed
oscarkey pushed a commit that referenced this pull request Nov 12, 2025
* Record copied public PR 562

* Fix build error on Python 3.9 (#562)

Our mac builds with Python 3.9 were all failing because an indirect dep stopped publishing packages for 3.9. See https://github.com/PriorLabs/TabPFN/actions/runs/18689219131/job/53367028472?pr=539

This fixes it by pinning away for the old builds.

(cherry picked from commit b215940)

---------

Co-authored-by: mirror-bot <mirror-bot@users.noreply.github.com>
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.

3 participants