lsst-sims / legacy_sims_catUtils

LSST Simulations package for catalog utilities
3 stars 4 forks source link

U/danielsf/debug/mba validation #81

Closed danielsf closed 7 years ago

danielsf commented 7 years ago

This pull request just adds some functionality that I noticed was missing while I was trying to validate our asteroid models. It adds a mixin that calls the chipNameFromPupilCoordsLSST() method, which is about a factor of six faster than chipNameFromPupilCoords(camera=LsstSimMapper().camera).

yoachim commented 7 years ago

Looks good.

On an unrelated note, testSN.py failed for me because pandas depreciated DataFrame.sort, so update line 470 to use .sort_values()