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

WIP: new plot for checking/investigating input ASTs #652

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

cmurray-astro
Copy link
Member

I wrote a new routine to plot some sanity-check level diagnostics for the input ASTs, ideally to be used (along with plot_ast_histogram) to make sure that the input ASTs look good before shipping them off to be processed. So far, this includes plotting their spatial distribution against the source density image (left, below) and, for all filters in the set, plotting CMDs as a function of source density (right, below; currently done in contours to simplify the plot, and entirely-overlapping contours is what we're looking for... i.e. all ASTs are replicated in the same way across all source density bins).

An example of what this looks like right now is below.

My questions are: is this useful? If so, what else should be included? Or how should the formatting be improved? Should I split this into individual plots (my original thought was to make a one-stop-shop for sanity checks).
15891_SMC-3956ne-9632_inputAST_diagnostic

@karllark
Copy link
Member

karllark commented Nov 2, 2020

I do think it is useful - thanks for opening this PR. I think it does require some documentation to explain the plots and what users should look for. Also, it would be good to get this into the main.py script in the base beast directory so that this script can be called with beast script_name.

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

Successfully merging this pull request may close these issues.

2 participants