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

Rename exp_time to exposure_time #47

Merged
merged 1 commit into from
Jan 18, 2022
Merged

Rename exp_time to exposure_time #47

merged 1 commit into from
Jan 18, 2022

Conversation

aboucaud
Copy link
Collaborator

No description provided.

@aboucaud aboucaud merged commit f27ff5a into main Jan 18, 2022
@aboucaud aboucaud deleted the exp_time_rename branch January 18, 2022 16:37
aboucaud added 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]>
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.

1 participant