
Hi all, It's bothered me for a long time that our image reading routines are hidden in scipy.misc. Here is a patch that implements image reading via the PIL array interface. Please let me know whether you like this or not. Regards Stéfan

On Thu, May 21, 2009 at 01:34:37AM +0200, Stéfan van der Walt wrote:
Hi all,
It's bothered me for a long time that our image reading routines are hidden in scipy.misc. Here is a patch that implements image reading via the PIL array interface. Please let me know whether you like this or not.
Docstring of imread seems wrong: the fname input argument is not a PIL image, it is the filename of the input image. Also, I think you should fail gracefully if PIL is not installed. RIght now this will give a traceback that might surprise the user ('ImportError: no module named PIL'). My 2 cents, Gaël

2009/5/25 Gael Varoquaux <gael.varoquaux@normalesup.org>:
On Thu, May 21, 2009 at 01:34:37AM +0200, Stéfan van der Walt wrote: Docstring of imread seems wrong: the fname input argument is not a PIL image, it is the filename of the input image.
Also, I think you should fail gracefully if PIL is not installed. RIght now this will give a traceback that might surprise the user ('ImportError: no module named PIL').
Thank you for the feedback, Gaël: http://projects.scipy.org/scipy/changeset/5765 Regards Stéfan
participants (2)
-
Gael Varoquaux
-
Stéfan van der Walt