Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 4, 2026

Describe your changes

Implements matrix testing strategy in CI pipeline to validate compatibility across Python versions:

  • Matrix dimensions:

    • Python: 3.12, 3.13, 3.14
    • Results in 3 parallel test jobs
  • Workflow changes (.github/workflows/create_test_conda_env.yml):

    • Added matrix strategy with fail-fast: false to test all Python versions
    • Python 3.14 marked as experimental with continue-on-error: true - failures won't block CI
    • GitHub notices display "Python 3.14 compatibility under development" for 3.14 jobs
    • Each job explicitly installs matrix-specified Python version after base environment creation
    • Job names include Python version for quick identification in CI

This enables detection of Python version-specific issues across the supported version range while allowing development to continue without blocking on potential Python 3.14 incompatibilities.

Issue ticket number and link (if applicable)

Related to investigation of CDO/python-cdo numerical differences with numpy>=2

Checklist before requesting a review

  • I ran my code
  • I tried to make my code readable
  • I tried to comment my code
  • I wrote a new test, if applicable
  • I wrote new instructions/documentation, if applicable
  • I ran pytest and inspected it's output
  • I ran pylint and attempted to implement some of it's feedback
  • No print statements; all user-facing info uses logging module

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Copilot AI changed the title [WIP] Create a matrix approach for scanning Python and NumPy versions Add matrix testing for Python 3.12-3.14 and numpy 1.26.*/2.4+ Feb 4, 2026
Copilot AI requested a review from ilaflott February 4, 2026 19:46
@codecov
Copy link

codecov bot commented Feb 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.77%. Comparing base (1b8e33c) to head (b927d1c).
⚠️ Report is 1 commits behind head on use-numpy-v2.

Additional details and impacted files
@@              Coverage Diff              @@
##           use-numpy-v2     #722   +/-   ##
=============================================
  Coverage         82.77%   82.77%           
=============================================
  Files                68       68           
  Lines              4529     4529           
=============================================
  Hits               3749     3749           
  Misses              780      780           
Flag Coverage Δ
unittests 82.77% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1b8e33c...b927d1c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ilaflott
Copy link
Member

ilaflott commented Feb 4, 2026

@copilot

  • ditch the matrix approach across numpy versions since it doesn't prevent the package from working with python3.12
  • make it so that the workflow using 3.14 doesn't have to succeed, and when it fails, have it throw a message like python3.14 compatibility under development

Co-authored-by: ilaflott <6273252+ilaflott@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 4, 2026

@copilot

  • ditch the matrix approach across numpy versions since it doesn't prevent the package from working with python3.12
  • make it so that the workflow using 3.14 doesn't have to succeed,...

Updated the workflow to:

  • Remove numpy matrix (now tests Python 3.12, 3.13, 3.14 only)
  • Added continue-on-error: true for Python 3.14 jobs so they don't block CI
  • Added GitHub notice "Python 3.14 compatibility under development" at the start of environment creation and in a final status check step

The workflow now runs 3 jobs instead of 6, focusing on Python version compatibility. (commit 65975cb)

Copilot AI changed the title Add matrix testing for Python 3.12-3.14 and numpy 1.26.*/2.4+ Add matrix testing for Python 3.12-3.14 with experimental 3.14 support Feb 4, 2026
Removed Python 3.14 from the testing matrix and related checks.
@ilaflott ilaflott marked this pull request as ready for review February 4, 2026 20:46
@ilaflott ilaflott merged commit a69aa60 into use-numpy-v2 Feb 4, 2026
6 checks passed
@ilaflott ilaflott deleted the copilot/sub-pr-716 branch February 4, 2026 20:46
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