Add linter

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.

[jarrodmillman]

Not sure folks are interested in this, but pre-commit has been pretty helpful for the other projects I work on. This does introduce some linting noise, but it is ignored by git and GitHub. I am happy to close this or modify it as folks see fit.

If you don't want to add this via pre-commit, I could make a separate PR that just updates the source files with pyupgrade and flynt to remove a ton of older Python cruft.

Thoughts?

[rgommers]

This seems like a useful set of changes to me. And I'm fine with adding the pre-commit yaml file. It doesn't seem to be triggering bot PRs, and the selected linter option seem okay (adding too many is a problem for usability, the ones here are okay with me).

I'll merge this soon unless there are any concerns. I haven't reviewed the full diff yet, but I assume they're all valid changes since they-re tool generated.

[jarrodmillman]

Just a reminder: This will need to be merged with a "Create a merge commit" for the .git-blame-ignore-revs commits to be valid.

[rgommers]

Would you mind fixing the merge conflicts?

[jarrodmillman]

@rgommers This should be ready to go. Is there anything else I can do to help with the next release?

[jarrodmillman]

@rgommers We decided to make pywavelets an optional dependency of scikit-image so we could get 3.12 wheels out:
https://pypi.org/project/scikit-image/0.22.0rc1/

But it would still be nice to get 3.12 wheels for pywt out soon. Let me know if I can help. I am happy to do release management for the next release if it would be helpful.

[jarrodmillman]

@rgommers Any updates on this or a new release? I am happy to handle the release if it would help. Thanks!!

[jarrodmillman]

@rgommers It would be great to get 3.12 binaries out soon. This is the only scikit-image (optional) dependency that doesn't support Python 3.12.

[rgommers]

Apologies for the delay @jarrodmillman . New release is up now. I'll deal with remaining PR reviews that were not needed for 1.5.0, including this one, afterwards (hopefully tomorrow).

[rgommers]

LGTM. I went through everything now - most of the diff is reordering of imports, moving to f-strings, and minor style changes. There were only a couple of lines to check more carefully; all looked fine and CI is green.

The new CI job required a bump and a small fix for other new code that was merged after this PR was opened. I don't expect it to be a burden; if it fails it should be obvious from the CI log what needs fixing. Development velocity on this repo is pretty low though; if the new lint job fails too much on average it can be disabled in the future (I expect it's fine though, since 5 months later only a few tweaks were needed).

In it goes! Thanks again Jarrod.