-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add additional filter curves #78
Conversation
Indeed, @dstndstn. @dkirkby is there a convention you would like me to follow with respect to the filter names? For example, for the intermediate-band filters would you like the prefix (and filter group) to be Also, is it possible for the same filter to belong to two different groups? For example, I would like to assemble a Other opinions? |
The API for loading a group of filters is load_filters, which is quite flexible since you can always specify an arbitrary list of filter names. It also supports a wildcard like "sdss-*" which is much less flexible. I suggest you group names by prefix in whatever way is likely to be the most useful default for users. The existing prefixes generally group bands that were in use on the same instrument at the same time. |
[WIP]
Additional filters are needed for a variety of scientific DESI analyses.