search

ChileanVirtualObservatory / acalib

Advanced Computing for Astronomy Library
http://acalib.readthedocs.io/en/latest/
GNU General Public License v3.0
7 stars 13 forks source link

MemoryError in loadFITS_PrimmaryOnly #55

Closed capgadsx closed 6 years ago

capgadsx commented 7 years ago

When using loadFITS_PrimmaryOnly() with big fits files (e.g. ALMA01002186_00_00_00.fits 1.8G) the function crashes with a MemoryError exception in the line: hduobject.data = (hduobject.data*bscale) + bzero

cvalenzu commented 7 years ago

Function name change to loadFITS_PrimaryOnly

maray commented 7 years ago

Why this function exists in the first place? The code is quite duplicated from HDU_to_NDData, so why not merging them?

capgadsx commented 7 years ago

The main difference is in how we open the FITS file, we can merge them to avoid duplicated code.