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

Make tqdm a required dependency? #1

Closed
shawnohare opened this issue Aug 9, 2019 · 1 comment
Closed

Make tqdm a required dependency? #1

shawnohare opened this issue Aug 9, 2019 · 1 comment

Comments

@shawnohare
Copy link

In a fresh Python 3.7.3 virtual environment with pdfmajor installed via pip, an attempt to follow the basic example in the documentation leads to a module not found error (cf. before) since tqdm is listed as a dev requirement.

Either the basic example aught to not utilize tqdm or it should be bundled as a required dependency, perhaps?

Python 3.7.3 (default, Jul 23 2019, 11:39:30)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.7.0 -- An enhanced Interactive Python. Type '?' for help.

[ins] In [1]: import pdfmajor

[ins] In [2]: from pdfmajor.interpreter import PDFInterpreter
---------------------------------------------------------------------------
ModuleNotFoundError                       Traceback (most recent call last)
<ipython-input-2-fbef3dceeb7a> in <module>
----> 1 from pdfmajor.interpreter import PDFInterpreter

~/.local/opt/pyenv/versions/3.7.3/envs/pdf/lib/python3.7/site-packages/pdfmajor/interpreter/__init__.py in <module>
      1 from typing import List
      2
----> 3 from tqdm import tqdm
      4
      5 from ..parser.PDFPage import PDFPage

ModuleNotFoundError: No module named 'tqdm'
@asosnovsky
Copy link
Owner

Thank you for reporting this.

Issue resolved under version 1.3.9

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

No branches or pull requests

2 participants