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

Do we need a camleot uninstalled in order to use this library? #191

Open
dejanmarkovic opened this issue Oct 19, 2024 · 2 comments
Open

Comments

@dejanmarkovic
Copy link

Hi team,

Thank you so much for maintaining this package!

I have a few questions though as I have not found those simple answers in the documentation.

  1. Do we need to uninstall a Camelot in order to use this library?
  2. What is the proper way to use this library

tables = **pypdf_table_extraction**.read_pdf(file_path, pages="all", flavor=network)
or
tables = **camelo**t.read_pdf(file_path, pages="all", flavor="stream")

  1. I have tried to use the flavor network or hybrid
    tables = pypdf_table_extraction.read_pdf(file_path, pages="all", flavor=network)
    and got the error 'An error occurred with stream flavor: name 'network' is not defined'.

Here is the output from the conda list
pypdf-table-extraction 0.0.2 pypi_0 pypi

I appreciate your feedback.
Thanks!

@bosd
Copy link
Collaborator

bosd commented Oct 20, 2024

Do we need to uninstall a Camelot in order to use this library?

Would like to know this as well. Currently I'm not using them simultaneously.
I only use the new library.

What is the proper way to use this library
Both the old name as the new one should work.

Afaik, the flavor argument should be formatted as a string with ""

@stefan6419846
Copy link

In theory you can install both libraries. Nevertheless, you will usually experience side effects: When importing from the camelot namespace, depending on the installation order the package content might differ. Additionally, uninstalling camelot will remove the camelot directory/module shipped by pypdf_table_extraction as well, possibly breaking existing code.

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

3 participants