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 BGS to cell 56 and 70 in minitest #403

Open
londumas opened this issue Aug 9, 2018 · 2 comments
Open

Add BGS to cell 56 and 70 in minitest #403

londumas opened this issue Aug 9, 2018 · 2 comments
Assignees

Comments

@londumas
Copy link
Contributor

londumas commented Aug 9, 2018

In minitest it would be useful to have the cell 56 and 70 for BGS also:
https://github.com/desihub/desitest/blob/master/mini/minitest.ipynb

@sbailey
Copy link
Contributor

sbailey commented Aug 10, 2018

Slightly more context: this is the plot in cell 70 that @londumas is requesting to have BGS included too:
unknown-4

Cell 56 is a plot of similar information, but generated by the minitest notebook itself so doesn't require any desisim QA code updates.

@sbailey
Copy link
Contributor

sbailey commented Aug 10, 2018

Cell 56 has been updated to include BGS in the latest notebook run:
blat

Would still be nice to have the deltav vs. z plots updated too to include BGS since that highlights more subtle biases and trends.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants