Skip to content
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

Closed
wants to merge 8 commits into from
Closed

add additional filter curves #78

wants to merge 8 commits into from

Conversation

moustakas
Copy link
Member

[WIP]

Additional filters are needed for a variety of scientific DESI analyses.

@dstndstn
Copy link
Contributor

dstndstn commented Jul 14, 2023 via email

@moustakas
Copy link
Member Author

The intermediate band filters are Suprime, not HSC, aren't they?

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 subaru-*, suprimecam-*, or something different?

Also, is it possible for the same filter to belong to two different groups? For example, I would like to assemble a cosmos2020-* filter group of 39(!) bandpasses, but many of them will belong in other groups. Or would you like the "groups" to be restricted to the same camera / telescope system?

Other opinions?

@dkirkby
Copy link
Member

dkirkby commented Jul 20, 2023

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.

@moustakas moustakas mentioned this pull request Sep 13, 2023
@weaverba137
Copy link
Member

As discussed in #79, we will close this PR and open a new PR that contains only the filter updates, since the test infrastructure updates are superseded by #79.

@moustakas moustakas deleted the more-filters branch October 3, 2023 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants