Skip to content

Fix warningfilter for deprecation in SciPy 1.12.0rc1#7275

Merged
mkcor merged 1 commit intoscikit-image:mainfrom
lagru:fix-warningfilter-scipy-deprecation
Dec 22, 2023
Merged

Fix warningfilter for deprecation in SciPy 1.12.0rc1#7275
mkcor merged 1 commit intoscikit-image:mainfrom
lagru:fix-warningfilter-scipy-deprecation

Conversation

@lagru
Copy link
Copy Markdown
Member

@lagru lagru commented Dec 22, 2023

Description

@scikit-image/core, we may want to get this small fix in so that we can fix PRs that are failing for SciPy 1.12.0rc1.

Not sure if the filters regex was wrong previously or if SciPy changed the message of the warning slightly. In any case, it began tripping up our test workflow using the --pre flag and SciPy 1.12.0rc1. E.g. see https://github.com/scikit-image/scikit-image/actions/runs/7299807113/job/19893319329

Checklist

Release note

Summarize the introduced changes in the code block below in one or a few sentences. The
summary will be included in the next release notes automatically:

...

Not sure if the filters regex was wrong previously or if SciPy changed
the message of the warning slightly. In any case, it began tripping up
our test workflow using the `--pre` flag and SciPy 1.12.0rc1 [1].

[1] https://github.com/scikit-image/scikit-image/actions/runs/7299807113/job/19893319329
@lagru lagru added the 🔧 type: Maintenance Refactoring and maintenance of internals label Dec 22, 2023
Copy link
Copy Markdown
Member

@mkcor mkcor left a comment

Choose a reason for hiding this comment

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

Ok, I'm not super clear on this, but this change can't hurt! Noticing, for example, under "Build package"

SciPy              1.12.0rc1

vs

scipy              1.11.4

@lagru
Copy link
Copy Markdown
Member Author

lagru commented Dec 22, 2023

I'm not following. I'm guessing that you are wondering why our CI is using SciPy 1.11.4, when this PR is talking about 1.12.0rc1? That would be because only the "linux-cp3.12-pre" workflow is using 1.12.0rc1.

@mkcor
Copy link
Copy Markdown
Member

mkcor commented Dec 22, 2023

No, no, sorry, I'm just noticing that there might be, indeed, some regex-related issues, when I read 'SciPy' (with uppercase 'S' and 'P') vs 'scipy' -- but maybe this change is unrelated to that fix!

(...) "linux-cp3.12-pre" workflow is using 1.12.0rc1.

Yes, I saw this. But I haven't seen the PRs failing for "linux-cp3.12-pre" -- where are they? Thanks!

@lagru
Copy link
Copy Markdown
Member Author

lagru commented Dec 22, 2023

E.g. the job from #7256 I linked above in the description.

@mkcor
Copy link
Copy Markdown
Member

mkcor commented Dec 22, 2023

Oh, thanks! (for reminding me I should review #7256 too 🙃)

@mkcor mkcor merged commit 0e35d80 into scikit-image:main Dec 22, 2023
@stefanv stefanv added this to the 0.23 milestone Dec 22, 2023
@lagru lagru deleted the fix-warningfilter-scipy-deprecation branch December 22, 2023 22:59
@lagru lagru mentioned this pull request Apr 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🔧 type: Maintenance Refactoring and maintenance of internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants