-
Notifications
You must be signed in to change notification settings - Fork 10
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
error with workers #6
Comments
Could you please paste me these below items?
|
Jasim, I'm using 2.7.5
as for the debug output, here's what i got:
if i instead run the conversion on a png i get a similar trace, but it ends with this recursive/endless |
Thanks @nealrs, Could reproduce the issue with non transparent |
From PIL's documentation
Its clear that, if we use a mask, it should be in mode "1", "L" or "RGBA" Could you please checkout the branch |
Jasim, I can embed non-transparent background images & text, but I'm still getting this with the new branch when I try a transparent image:
|
i did have success with a non-transparent gif |
When I almost try to reproduce the situation, the system works smooth :(
Could you please paste a DEBUG log of this error? |
Strange!!! |
#6 Muliprocessing, Serialization issue
There were some serialization issues with PIL image files, passed to multiprocessing job. Implemented pickle serialization and everything is smooth now |
thanks @jasimmk, looks like that fixed the errors -- one more quick question though: how is the watermark sized & is there a ratio/pixel density I should use? |
Hi @nealrs , Closing this bug now. As of now, the watermark size is calculated based on relative size of common watermarks. If you find it very handy, that we need to control size of watermarks, We could add it as new feature. Could you please comment to this #12 [Command line argument to control watermark size] ticket?. |
Hey Jasim, I'm having a little trouble with your script. I've successfully used the text watermarking, but whenever i try to watermark with an image, i get this error (and I do have pillow & colour installed):
The text was updated successfully, but these errors were encountered: