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

I wasn't able to get camelot working without installing ghostscript in my python environment as well #550

Closed
wants to merge 1 commit into from

Conversation

sohanglal
Copy link

Therefore I added the following command to the documentation : pip install ghostscript
This was despite having ghostscript installed system wide

@bosd
Copy link
Collaborator

bosd commented Mar 3, 2025

This extra step, should no be needed. Which version did you try to install?

@DoomedJupiter
Copy link
Collaborator

I had a similar issue, not with ghostscript but with other packages, so I'm adding a comment here. I wrote this up as an issue in the previous repo here before realizing this project migrated. I will happily recreate the issue over here if desired @bosd.

@sohanglal
Copy link
Author

p camelot.__version__
'1.0.0'

I tried it on 2 operating systems: NixOS and Ubuntu. Both in which ghostscript was installed via the native package manager and camelot-py was installed using uv.

@bosd
Copy link
Collaborator

bosd commented Mar 4, 2025

This seems to be a package manager / toml related issue.
I don't think the proposed solution here to update the docs is the proper answer.

Better, to dig a bit deeper to find the root cause of these issues and fix them, instead of this workaround.

It might be related that we switched from poetry to uv.

@DoomedJupiter
Copy link
Collaborator

@bosd Did some additional digging on this topic and wrote up an issue. Long story short, looks like adding pillow as a dependency for camelot would resolve this issue. I'm not familiar with the inner workings of the package though, or what poetry/uv is so maybe its something else.

@bosd
Copy link
Collaborator

bosd commented Mar 4, 2025

@DoomedJupiter Thanks for the very detailed write-up.
Adding pillow as a dependency seems the way forward.
So closing this one.

@bosd bosd closed this Mar 4, 2025
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.

3 participants