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

Update resize.py #7

Merged
merged 1 commit into from
Jan 31, 2014
Merged

Update resize.py #7

merged 1 commit into from
Jan 31, 2014

Conversation

minosniu
Copy link
Contributor

Wrong import format for PIL.Image

Wrong import format for PIL.Image
@Zulko
Copy link
Owner

Zulko commented Jan 29, 2014

Thanks. On my computer import Image doesn't raise any error, do you know what the differences with your change are ? Did you get a bug with the previous line ? Are you sure your changes lead to the same behaviour ?
Sorry for the questions, I am not a PIL expert.

@minosniu
Copy link
Contributor Author

My laptop is MacOS Maverick, pip installed PIL. Without "From PIL import Image" it would raise an exception at every .resize()

I'm not an expert of PIL, either. It's a dying library anyway.

With my change does it run on your setup?

@Zulko
Copy link
Owner

Zulko commented Jan 29, 2014

Both imports seem to work in python but I am still at work. I make a test when I am home, just to be sure, and I merge the request.

@Zulko
Copy link
Owner

Zulko commented Jan 31, 2014

Thanks.

Zulko added a commit that referenced this pull request Jan 31, 2014
@Zulko Zulko merged commit 8a827fd into Zulko:master Jan 31, 2014
ghost pushed a commit that referenced this pull request Feb 20, 2017
sync with zulko/moviepy master
ghost pushed a commit that referenced this pull request Feb 23, 2017
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.

None yet

2 participants