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

New SV selection for QSO with DR9 #652

Merged
merged 1 commit into from Nov 20, 2020
Merged

New SV selection for QSO with DR9 #652

merged 1 commit into from Nov 20, 2020

Conversation

Cyeche
Copy link
Contributor

@Cyeche Cyeche commented Nov 19, 2020

Tuning of the QSO SV selection for DR9
A few plots to illustrate the new selection
All_QSO

SV_QSO_Selection

@geordie666
Copy link
Contributor

Thanks, @Cyeche: This all looks good from the perspective of the code. I'll merge it later today.

@geordie666
Copy link
Contributor

@Cyeche: I forgot, the usual requests apply for all changes to targeting cuts. Please:

(1) Update the wiki pages where appropriate with the new cuts.
(2) Update the changes.rst file in doc with the PR information.

I'll take care of (2) in my current working branch of desitarget, but please update the wiki.

@geordie666 geordie666 merged commit f67c3bc into master Nov 20, 2020
@geordie666 geordie666 deleted the QSO_SV_for_DR9 branch November 20, 2020 01:14
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.

None yet

2 participants