-
Notifications
You must be signed in to change notification settings - Fork 85
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
Science Review for PSF Photometry Notebook #20
Comments
Ugh, Ignore the previous post (accidentally pressed Enter and github has no real delete function). I originally sent these comments to Matteo on slack, and he said the typical review process involves opening a ticket on github, which I assume meant issue. Since I originally sent everything on Slack, it's not particularly well formatted, but here are my comments:
-I think another useful thing to show may be the outputs of the PSF fitting, as in the catalogs before they go into the CMD's, just so users know what information is available. Comparing fits to groups containing multiple stars vs a single star, or even just commenting on the behavior there could be useful. -The last science related comment I have would be to also give some comment about the astrometric information that comes out of the fits. The benefits of the PSF fitting are often great when looking for accurate positions, and typically better than DAOFind type routines in my experience. A potential way to show this could be to plot the separations between the fit positions of the same stars in the two bands vs mag, to get some idea of astrometric precision. I think that will be very interesting to those looking to measure proper motions/detect microlensing etc. -Last thing: for readability, putting a lot of the stuff that gets ran twice in a cell into a function could be nice. For instance the detection of stars, the fitting, and the matching of the stars can likely be condensed into functions, and just have lines later that call them. This is also nice because users can copy/paste the relevant functions right out for their own analyses. |
No description provided.
The text was updated successfully, but these errors were encountered: