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

compatibility with new version of pdfminer-20131113 #1

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

medphisiker
Copy link

hello, I download pdftables 0.3

In new version of pdfminer (pdfminer-20131113) PDFDocument have new location
it's location now in
from pdfminer.pdfdocument import PDFDocument

In pdftables.py

insead of string:
from pdfminer.pdfparser import PDFParser, PDFDocument

you could write :

from pdfminer.pdfparser import PDFParser

try:
from pdfminer.pdfparser import PDFDocument
except ImportError:
from pdfminer.pdfdocument import PDFDocument

@jtbates
Copy link

jtbates commented May 30, 2014

I cloned and installed the the development version and it doesn't seem to have this problem. Installing with pip does not work as it is an older version. @drj11 - can you update the package on PyPI? Not only is it a problem for using pdftables on its own, the messytables package also depends on it.

@davidastephens
Copy link

The development version doesn't have the problem because it's using poppler as the default backend. I'll send pull request that fixes the pdfminer api change in case anyone wants to use that as the backend.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants