mommermi / photometrypipeline

automated photometry pipeline for small to medium-sized observatories
GNU General Public License v3.0
62 stars 26 forks source link

Pillow now required? #18

Closed boada closed 7 years ago

boada commented 7 years ago

The latest merge breaks my installation.

Traceback (most recent call last): File "/home/boada/Projects/photometrypipeline/pp_run", line 46, in import pp_prepare File "/home/boada/Projects/photometrypipeline/pp_prepare.py", line 44, in import diagnostics as diag File "/home/boada/Projects/photometrypipeline/diagnostics.py", line 38, in from scipy.misc import toimage # requires Pillow ImportError: cannot import name toimage

It seems this is not an optional requirement. Pillow is not listed as a required install in the documentation.

mommermi commented 7 years ago

Yes, I made a number of changes last week, one of which was to use pillow throughout diagnostics.py as I was running into segmentation faults using 20kx20k images. I haven't had time to update the documentation but will do this as soon as possible.

boada commented 7 years ago

I've started running into these strange segmentation faults. My images are: NAXIS1 = 14793 / NUMBER OF ELEMENTS ALONG THIS AXIS
NAXIS2 = 14571 / NUMBER OF ELEMENTS ALONG THIS AXIS

Guess I am going to need to update. Lame-o