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

QA with Truth (S/N and redshift refactor) #254

Merged
merged 13 commits into from
Aug 14, 2017
Merged

QA with Truth (S/N and redshift refactor) #254

merged 13 commits into from
Aug 14, 2017

Conversation

profxj
Copy link
Contributor

@profxj profxj commented Aug 14, 2017

New code for generating QA (plots) using the Truth.
Two areas are S/N plots (as a function of object type)
and refactoring redshifts.

Here are the main changes:

  • Add desi_qa_s2n script
  • Refactor desi_qa_zfind script
  • Refactor redshift QA for new data model
  • Refactor shared QA methods to desisim.spec_qa.utils
  • New plots for S/N of spectra for various objects (ELG, LRG, QSO)

Here are some examples of the S/N and new redshift plots:

dc17a2_s2n_elg_z

dc17a2_s2n_lrg

tst_lrg

@sbailey
Copy link
Contributor

sbailey commented Aug 14, 2017

Looks good. We'll have some future tweaks but can merge now to get the updated data model structure in place.

I'm curious why the faintest LRG bin had higher S/N than the second faintest bin. Low statistics fluctuation? Double observation of the faintest and single observation of the sort-of-faint ones? We should adjust the mag bin boundaries to match the 1-,2-,3-observation LRG (soon to be split out into separate targeting bits). Currently the boundaries are

  • zmag < 20.36 : 1 obs
  • 20.36 <= zmag < 20.56 : 2 obs
  • 20.56 <= zmag : 3 obs

I'm sure those numbers will be adjusted in the future and we shouldn't arbitrarily hard code them here, so this is mainly a comment as a heads up for future consideration.

@sbailey sbailey merged commit d13f0f8 into master Aug 14, 2017
@sbailey sbailey deleted the s2n_galaxy branch August 14, 2017 16:58
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.

2 participants