LSSTDESC / skyCatalogs

Create sky catalogs and provide access via API
BSD 3-Clause "New" or "Revised" License
6 stars 5 forks source link

Rationalize options for sc_create.py #117

Closed JoanneBogart closed 2 months ago

JoanneBogart commented 3 months ago

Several changes should be considered:

JoanneBogart commented 3 months ago

@jchiang87 do you have any comments about any of these? Is there anything more that belongs on this list?

jchiang87 commented 3 months ago

As far as the options themselves are concerned, I don't have any additional comments. However, even though this is outside the scope of just changing the options, I'd recommend splitting the flux file generation off as an entirely separate application, since that really only depends on the main files, and I would think that most of the options would end up being irrelevant in this case.

JoanneBogart commented 2 months ago

Addressed by PR #120