gammapy / gamma-cat

An open data collection and source catalog for gamma-ray astronomy
https://gamma-cat.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
15 stars 17 forks source link

Issue 219 no2 #240

Open pdeiml opened 6 years ago

pdeiml commented 6 years ago

This is one way how to control which SED goes to the catalog.

If you look to input/gammacat/gamma_cat_sed.yaml and there at source_id = 1, you can see the idea:

I only want to introduce this idea of a schema and if it is agreed then one needs to add the keywords to all sources.

cdeil commented 6 years ago

I would stronly prefer to only have one single config file for the catalog. I.e. my suggestion would be to change this file https://github.com/gammapy/gamma-cat/blob/master/input/gammacat/gamma_cat_dataset.yaml to add the extra info, e.g. a sed_ref_id key.

pdeiml commented 6 years ago

Ok, if we stay with one single file, then I would suggest to add three additional infos: sed_ref_id lc_ref_id ds_ref_id (data set)

What do you think?

cdeil commented 6 years ago

I would leave reference_id as-is, and optionally if there is reference_id_sed or reference_id_lc, use that for SED or LC (and otherwise default to use the same for all). I.e. minimal edits to get what we want.

But either way, your more verbose, but also more explicit choice for each part of the catalog data is also good.