search

megalut / sewpy

Source Extractor Wrapper for Python
GNU General Public License v3.0
22 stars 11 forks source link

Added functionality for writing default.psf and accepting user specified input psf file #15

Closed aaryapatil closed 8 years ago

aaryapatil commented 8 years ago

The ability to write all the default files in the working directory, makes sewpy way more useful. Unfortunately the default.psf file hasn't yet been added to this, hence I added functionality for writing default.psf and accepting user specified input psf file. This removes the sextractor Error: PSF file not found: default.psf, which you will face if you don't have a psf file in the current directory.

mtewes commented 8 years ago

Hi @AustereCuriosity and thanks for the nice additions! It all looks fine, and I'll merge it in right now. I'll open a related issue about testing this, just to remind ourselves about these changes the next time we use sewpy. Astropy now becomes a dependency, just for this minor feature of writing the default.psf file (done everytime sewpy is run). I've never used SExtractor PSF-fitting, and I can't judge how worthy it is to have this default.psf file in there, but I do understand the point of "shipping" it with sewpy.

aaryapatil commented 8 years ago

Thanks for the merge. I faced this issue while I was using sewpy to get MAG_MODEL as one of the output paramters which requires PSF-fitting. Hence I thought to do the same.