Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Replace FT2Image by plain numpy arrays.#30044
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
Failures look related and real, but I have no good guess as to why not on osx. |
fc3e947
toc69331b
CompareI forgot to zero-initialize the FT2Image. Fixed. |
Uh oh!
There was an error while loading.Please reload this page.
QuLogic commentedMay 17, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
If we end up doing#30059, does that mean this PR is also not necessary (other than applying a deprecation, probably)? |
Indeed, but I didn't remove this PR as 1) the deprecation will still be needed anyways, and 2) it's a much simpler PR, which can hopefully get in faster if#30059 gets stalled. |
Uh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
def8fa4
intomatplotlib:mainUh oh!
There was an error while loading.Please reload this page.
FT2Image predates numpy, but we don't really need to maintain our own 2D buffer class nowadays.
PR summary
PR checklist