Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the deprecation error for resize_event #158

Merged
merged 1 commit into from
Feb 8, 2024

Conversation

monodera
Copy link
Contributor

@monodera monodera commented Feb 7, 2024

resize_event has been deprecated since matplotlib 3.6. This commmit transitions from resize_event to ResizeEvent as recommended.

Ref: https://qiskit.org/ecosystem/metal/_modules/matplotlib/backend_bases.html

Fixes: #157

@monodera
Copy link
Contributor Author

monodera commented Feb 7, 2024

I'm not sure some backward compatibility needs to be implemented.

@olebole
Copy link
Member

olebole commented Feb 7, 2024

Thank you very much for the bug report and the PR!

From a quick search on the Matplotlib documentation I get the impression that the ResizeEvent is supported since ages; at least it is mentioned in their pre-historic changelog in 2005.

From this, I would not see the need for further backward compatibility, but may someone test with a relevant Matplotlib version?

@olebole
Copy link
Member

olebole commented Feb 7, 2024

@monodera I updated the main branch to fix the readthedocs (and other CI) errors. Could you rebase your branch to the updated main?

@olebole olebole changed the title This commit fixes the deprecation error for resize_event (#157). Fix the deprecation error for resize_event Feb 7, 2024
…unity#157).

`resize_event` has been deprecated since matplotlib 3.6. This commmit
transitions from `resize_event` to `ResizeEvent` as recommended.

Ref: https://qiskit.org/ecosystem/metal/_modules/matplotlib/backend_bases.html
@monodera
Copy link
Contributor Author

monodera commented Feb 7, 2024

@monodera I updated the main branch to fix the readthedocs (and other CI) errors. Could you rebase your branch to the updated main?

done!

@olebole olebole merged commit e41ad9f into iraf-community:main Feb 8, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

MplCanvasAdapter error with a pyraf window with matplotlib graphics
2 participants