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

[Don't merge] Swap out gold sample for APOGEE DR17 unimodal sample (beta) #12

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adrn
Copy link
Collaborator

@adrn adrn commented Mar 2, 2021

This is an initial version of what will be the gold sample for the upcoming APOGEE data release (DR17). The MAP parameter values may be a little off here because I had some issues getting the MCMC sampling to work for all of the systems, but it is ~2x as many sources as the previous catalog, so I'm interested to see how things look! Since this is a beta version, we shouldn't switch to this for production, but hoping that CI will produce the figure I want to see 🙏?

@adrn
Copy link
Collaborator Author

adrn commented Mar 2, 2021

Ah ok maybe CI only runs the style check on PRs, so this won't work. Oh well!

@dfm
Copy link
Member

dfm commented Mar 2, 2021

Alas none of these figures get generated via CI yet (it's currently implemented in Untitled32.ipynb...) I'll try this or share the relevant data file soon!

@dfm
Copy link
Member

dfm commented Apr 16, 2021

Here's the xmatch for this sample (note that the normalized residuals histogram does not include the reported APOGEE uncertainties):

apogee-gold

@adrn
Copy link
Collaborator Author

adrn commented Apr 16, 2021

beautiful!

@adrn
Copy link
Collaborator Author

adrn commented Apr 16, 2021

PS I will have a new, "official" DR17 unimodal sample soon (weeks)

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