-
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.18.14 changed capitalization of deprecated getTextlength to getTextLength (capital L) #1085
Comments
Downstream example: https://gitlab.com/plom/plom/-/merge_requests/948 where we currently use a minimum value of
We either: bump our min value but then distro users must upgrade their pymupdf (which is honestly not that big a deal! :-) Anyway, let me know if this was intentional and we'll bump of min dep or if accidental and can expect a 1.18.15? |
By no means intentional! On the contrary, I am aware this is an unpleasant change overall, so I want to make it as smooth and gentle as possible Python 3.8.5 (default, Jan 27 2021, 15:41:15)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.23.1 -- An enhanced Interactive Python. Type '?' for help.
In [1]: import fitz
In [2]: fitz.getTextLength("hello")
Out[2]: 23.23199987411499
In [3]: print(fitz.__doc__)
PyMuPDF 1.18.14: Python bindings for the MuPDF 1.18.0 library.
Version date: 2021-06-01 08:11:38.
Built for Python 3.8 on linux (64-bit).
In [4]: |
As we discussed earlier: Deprecation warning will start being issud in version 1.19.0. |
how strange! I will dig.... |
I figured it out!
|
WTF - a typo in the alias definition! |
Fixed in new version. |
In 1.18.12 and 1.18.12:
But in 1.18.14:
This is the usual problem we've discussed before: its hard to downstream users to have a minimum supported version while also supporting the latest version.
The text was updated successfully, but these errors were encountered: