jennykim1016 / OM10

Tools for working with the Oguri & Marshall (2010) mock catalog of strong gravitational lenses
MIT License
0 stars 0 forks source link

Overwrite `qso_mock.fits` #7

Closed jennykim1016 closed 7 years ago

jennykim1016 commented 7 years ago

I should overwrite qso_mock.fits file so that it contains the columns of synthetically colored magnitudes. This will save a lot of computation time.

drphilmarshall commented 7 years ago

For now, you can just make a new file, qso_mock_ugrizy.fits that contains ugrizy magnitudes of the lens galaxies and source qsos. Once it's validated we can check it in - but not before!

jennykim1016 commented 7 years ago

Oh, I checked your comment just now. I saved in a new file, qso_mock_colored.fits. In addition, db.py threw an error whenever I wanted to right down the db.sample data to .fits file using writeto method. It was because db.sample was not in a form of numpy array(rather, it was in the form of astropy.table), so I added one line where I change db.sample to the numpy array before saving it.

drphilmarshall commented 7 years ago

Interesting. Were you able to read it back in from your new file, using a new db object?

On Wed, Feb 8, 2017 at 3:05 PM, Jenny Kim notifications@github.com wrote:

Oh, I checked your comment just now. I saved in a new file, qso_mock_colored.fits. In addition, db.py threw an error whenever I wanted to right down the db.sample data to .fits file using writeto method. It was because db.sample was not in a form of numpy array(rather, it was in the form of astropy.table), so I added one line where I change db.sample to the numpy array before saving it.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/jennykim1016/OM10/issues/7#issuecomment-278492071, or mute the thread https://github.com/notifications/unsubscribe-auth/AArY9yqE68qWlI9vF1Memq5DJe_4ioucks5rakpRgaJpZM4L29-M .

jennykim1016 commented 7 years ago

Yes! I was just about to clean up all the notebooks and comment more, but it works.

screen shot 2017-02-08 at 5 14 06 pm

p.s: importing fits is an unnecessary line. I am going to erase it.

drphilmarshall commented 7 years ago

Super!

Also, nice screenshot - most helpful :-)