-
Notifications
You must be signed in to change notification settings - Fork 8k
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
Update quickstart.md for a better python pdf demo #11927
Conversation
fix issues: 1.getPixmap() function is not recognized,changing to get_pixmap 2.fix TypeError when paddle recognized an empty page 3.pre-stored pageCount to avoid issues 4.added GPU usage
Thanks for your contribution! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
getPixmap
is deprecated. see :
https://pymupdf.readthedocs.io/en/latest/znames.html#deprecated-names
啊这,突然发现我只更新了中文文档,一会我还会再提个英文文档的修改,麻烦了 |
feel free to assign me to review. 🚀 |
@qwedc001 Thanks for your contribution! You will receive a beautiful PaddlePaddle gift. Please provide your mailing address by filling out the following questionnaire before October 18th. Looking forward to the future, we will walk further together in the world of open source! |
Thanks @luotao1 much appreciated. |
hi, @qwedc001
|
fix issues:
1.getPixmap() function is not recognized,changing to get_pixmap
2.fix TypeError when paddle recognized an empty page
3.pre-stored pageCount to avoid issues
4.added GPU usage
getPixmap好像是个别名函数,我不知道是我配置的问题还是新版本弃用了,在我这里只有get_pixmap能用,可能要麻烦你们再测试一下
然后识别到空页会直接退出弹TypeError有点太难绷了,于是就有了这个pr(