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

U/danielsf/validate centroid #392

Open
wants to merge 44 commits into
base: master
Choose a base branch
from
Open

Conversation

danielsf
Copy link
Contributor

This PR adds a script compare_centroids.py which reads in a centroid file and an InstanceCatalog. It uses CatSim to predict the pixel positions of the sources in the InstanceCatalog. It then generates a .tex file containing some scalar values and plots that summarize the agreement between the PhoSim pixel positions and the CatSim-predicted pixel positions of the sources in the InstanceCatalog. This .tex file can be compiled into a .pdf using pdflatex.

this is necessary for us to go from RA, Dec in a PhoSim InstanceCatalog
to actual RA, Dec
this is for generating temporary files used by unit tests
output weighted, mean, and median dx/dy to stdout
also: mangle figure names to avoid overwriting older figures
@danielsf
Copy link
Contributor Author

@TomGlanzman I added a method to return a dict of scalar summary statistics. Let me know if you want anything else.

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.

3 participants