-
Notifications
You must be signed in to change notification settings - Fork 510
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
1.17.6: insertImage: rect cannot be a list #622
Comments
I guess I'm not allowed to pass a list for This works:
|
Yes, you ought to be! |
This is a workaround for pymupdf/PyMuPDF#622 which may not even by a bug b/c in other places we use `fitz.Rect`. So its certainly not wrong to use `fitz.Rect` explicitly!
Thanks! Docs say Actually in most places my co-author had uses [1] update: by "docs" I mean docstring of |
No, no, it was my bad. The intention is to allow |
@cbm755 - concerning gluing / glueing: my dictionaries tell me that both forms are correct. Which I didn't know either. |
I see. I put that one as a single separate commit b/c I was unsure so feel free to revert it. |
Never mind, really. I am glad you have been looking through major parts of the docu, again a big thank you for this. |
New version 1.17.7 addressing this is being uploaded right now. |
This is a workaround for pymupdf/PyMuPDF#622 which may not even by a bug b/c in other places we use `fitz.Rect`. So its certainly not wrong to use `fitz.Rect` explicitly!
On upgrade to 1.17.6, I started getting:
configuration
Minimal example:
The text was updated successfully, but these errors were encountered: