search

matthewearl / faceswap

Python script to put facial features from one face onto another
MIT License
1.4k stars 496 forks source link

Enhance type-conversation #3

Closed sinwoobang closed 9 years ago

sinwoobang commented 9 years ago

Hi. I got rid of unnecessary type-conversations.

Could you tell me your thought?

Thanks.

matthewearl commented 9 years ago

Thanks for your interest! I intentionally wrapped range() with list() for the purposes of Python 3 compatibility: On Python 3 range() returns a generator, so can not be read more than once. (Note I haven't tried the script on Python 3, so there may be other incompatibilities, but keeping the list() calls seems like a step in the right direction.)