Skip to content

fix: standard-imghdr is removed from standard library in 3.13#705

Merged
cako merged 4 commits into
PyLops:devfrom
cako:update-313-docs
Oct 24, 2025
Merged

fix: standard-imghdr is removed from standard library in 3.13#705
cako merged 4 commits into
PyLops:devfrom
cako:update-313-docs

Conversation

@cako
Copy link
Copy Markdown
Collaborator

@cako cako commented Oct 22, 2025

No description provided.

@cako cako requested a review from mrava87 October 22, 2025 02:59
@mrava87
Copy link
Copy Markdown
Collaborator

mrava87 commented Oct 22, 2025

@cako where do we rely on this?

@cako
Copy link
Copy Markdown
Collaborator Author

cako commented Oct 23, 2025

We don't right now! This is for if we want to ever build docs in 3.13 and above. I was testing on 3.13 and noticed this issue of the library being dropped from stdlib.

@mrava87
Copy link
Copy Markdown
Collaborator

mrava87 commented Oct 23, 2025

Oh right! Well, I am all in favor of keeping the python version high in CI and doc (CI is already supporting 3.13, soon I'll try to see what happens if I add 3.14 😉 )... let me try

@mrava87
Copy link
Copy Markdown
Collaborator

mrava87 commented Oct 23, 2025

Mmh 3.13 fails on intel_mkl: https://app.readthedocs.org/projects/pylops/builds/30058430/

Copy link
Copy Markdown
Collaborator

@mrava87 mrava87 left a comment

Choose a reason for hiding this comment

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

Ok, looks like we can't go up to 3.13 yet, but if you agree I am happy to keep your change and bump to 3.12 for now?

@cako
Copy link
Copy Markdown
Collaborator Author

cako commented Oct 24, 2025

@mrava87 sounds good! I'll merge this ones for now

@cako cako merged commit 21033fd into PyLops:dev Oct 24, 2025
20 checks passed
@cako cako deleted the update-313-docs branch October 24, 2025 02:42
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