Skip to content

[rocprofiler-compute] Bump scipy test pin for Python 3.13 support#7784

Open
vedithal-amd wants to merge 1 commit into
developfrom
users/vedithal-amd/bump-scipy-py313
Open

[rocprofiler-compute] Bump scipy test pin for Python 3.13 support#7784
vedithal-amd wants to merge 1 commit into
developfrom
users/vedithal-amd/bump-scipy-py313

Conversation

@vedithal-amd

@vedithal-amd vedithal-amd commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Motivation

scipy 1.11.4 fails to install on Python 3.13: it pins legacy Cython <3.0, which cannot compile against the CPython 3.13 C-API. Bump the test pin to scipy 1.14.1, keeping numpy 1.26.4 unchanged.

Technical Details

  • requirements-test.txt: scipy 1.11.4 -> 1.14.1.
  • 1.14.1 is the lowest scipy with a prebuilt cp313 wheel and runs against the existing numpy 1.26.4 pin.

JIRA ID

ROCM-26919

Test Plan

  • Create a Python 3.13 venv, run pip install -r requirements-test.txt, then run the test suite.

Test Result

Verified on Python 3.13: scipy 1.14.1 installs against the existing numpy 1.26.4 pin, with no change to test behavior.

Submission Checklist

- scipy 1.11.4 pins Cython <3.0, which fails to build on CPython 3.13.
- Bump to scipy 1.14.1, the first release with a cp313 wheel; numpy 1.26.4 stays.

Co-Authored-By: Claude Opus 4 (1M context) <noreply@anthropic.com>
@vedithal-amd vedithal-amd requested a review from a team as a code owner June 24, 2026 21:50
Copilot AI review requested due to automatic review settings June 24, 2026 21:50

Copilot AI left a comment

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.

Pull request overview

This PR updates rocprofiler-compute’s test-only Python dependency pin to ensure SciPy installs successfully on Python 3.13 (where SciPy 1.11.4 fails to build/install).

Changes:

  • Bump scipy in requirements-test.txt from 1.11.4 to 1.14.1 (keeping the runtime numpy==1.26.4 pin unchanged in requirements.txt).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants