Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: daf_butler, obs_decam
-
Labels:
-
Story Points:1
-
Epic Link:
-
Sprint:AP S21-3 (February)
-
Team:Alert Production
-
Urgent?:No
Description
At present, the physical_filter for DECam is something like "g DECam SDSS c0001 4720.0 1520.0", which is not ideal. It should probably just be "g"? Or perhaps, e.g., "g_4720_1520" if more specificity is desired. (And so on for the other bands.)
The goals here are (1) no spaces and (2) shorter as feasible.
Ideally this can be fixed in the next few weeks, before any shared DECam gen3 repository lands in /datasets.
Attachments
Issue Links
- relates to
-
- Done
Activity
Nobody is ever going to have both "U 2" and "U_2" as physical filters, are they? (Of course, now that I say it, someone is going to do it...) We should document that this is banned.
I hope they don't. It's the balance between usability of data products and uniqueness of dimension space.
Haha... I cannot imagine this reality, but that's how many weird breakages happen, isn't it. I'd be happy to document this is banned if someone tells me where to do so.
That looks like it should work (I didn't see a pull request so can't comment on github). It would be good if you could add a test to test_templates.py – maybe as simple as turning the physical_filter in the example into "U 2" or something.