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

Adding missing gain values #44

Merged
merged 1 commit into from
Jan 14, 2022
Merged

Adding missing gain values #44

merged 1 commit into from
Jan 14, 2022

Conversation

mpaillassa
Copy link
Collaborator

This PR is related to issue #39 and aims to add the gain values for all surveys.

@aboucaud aboucaud linked an issue Jan 14, 2022 that may be closed by this pull request
@aboucaud
Copy link
Collaborator

Looking good, thanks @mpaillassa

@aboucaud aboucaud merged commit fbe54eb into LSSTDESC:main Jan 14, 2022
aboucaud pushed a commit that referenced this pull request Feb 9, 2022
aboucaud added a commit that referenced this pull request Feb 13, 2022
* Add data README

* Add file to package data

* Add link to main README

* Address Maxime comment

* Update DES references (#42)

* Adding missing gain values (#44)

* Adding missing obscuration values (#46)

* Rename exp_time to exposure_time (#47)

* Update pre-commit

* Update sky brightness definition

* Fix exposure_time name

* CFHT references (#51)

* first rework
* cfhtls exposure times
* renaming to cfhtls

* Include the effective wavelength

* Add ismael's remarks

Co-authored-by: mpaillassa <[email protected]>
Co-authored-by: mpaillassa <[email protected]>
@mpaillassa mpaillassa deleted the gain branch February 14, 2022 07: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.

Add gain information
2 participants