-
Notifications
You must be signed in to change notification settings - Fork 30
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
dN/dmag test #7
Comments
I am going to start working on this. |
Here's a preliminary result from this test, made by @duncandc: (and the corresponding entry on DESCQA web interface) Lines are protoDC2 and buzzard_test, points are HSC. Difference in normalization may be due to a bug (still be explored). See also LSSTDESC/DC2-production#21 for related comments. |
@yymao @duncandc - There has been further discussion of this test in #50 (see specifically from #50 (comment) onwards). A few outstanding questions:
|
@rmandelb To answer your questions ---
|
The criteria Rachel suggests sounds reasonable to me. |
I am testing my calculation of dn/dmag. Here is a comparison between my calculation and the result in the HSC data release paper. This suggests that there is a problem with the test. But for the life of me I can not find the problem. Particularly confusing is the offset in the peak where incompleteness sets in. the magnitudes are offset by ~1. [compare lime green line and points] |
@duncandc is there any chance that there's a missing 5*log(h) ~= -0.77? |
apparent magnitudes don't scale with h, right? |
no, i don't think they do |
everyone should feel free to take a look at the notebook in the desqagen branch of my fork. https://github.com/duncandc/descqa/tree/descqagen/descqagen/app_mag_func_test |
Apparent magnitudes have a contribution from the luminosity distance 5*np.log10(dl)+25 which assumes dl is in Mpc. But the catalog providers take care of this. What about magnitude definitions? Catalog fluxes are converted to magnitudes in the AB system. Does this match the definition in Hiroki et al? |
@evevkovacs the most recent plot @duncandc showed is comparing the HSC data with HSC paper plot. So there seems to be something going on in Duncan's data reduction but we haven't figured out what it is. It does seem that the dN/dmag in the HSC paper is closer to the catalogs' |
I think I see the problem in the notebook. YOu have a variable representing the full area, but you seem to have assigned its value based on the area of just 1 region:
So your "full area" is too small by a factor of ~5. You later use this area to normalize densities for the whole catalog. |
@rmandelb, that mostly does the trick. |
Is this protoDC2 or Buzzard that is below the data release paper? |
To be clear, the reason i ask is that we know about a few effects:
Both of these effects have the right sign at least, but the severity and magnitude-dependence differs for the two effects. |
Although, it just occurred to me that I got these points from plot digitizer... so I don't know how much trust we put in my steady hand... |
This is a comparison between my measurement of dn/dmag and the data presented in figure 16 in Aihara+ 2017 (https://arxiv.org/abs/1702.08449). |
Well, I would be comfortable at this point going back to your dn/dmag estimate from HSC vs. the sims... |
Is the legend really correct? I believe the number counts in HSC should really flatten out beyond i=26, but the green points labeled HSC keep rising all the way to i=30. |
OK. Well, I think the suppression at protoDC2 beyond i=23 has to do with the z=1 limit, which is particularly important there. I had made a suggestion somewhere for how to account for this (basically taking the DEEP2 dN/dz for mag-limited samples, and using that to "correct" for missing galaxies), but perhaps for now we should do comparisons with Buzzard? Also, can you by default have a separate panel for the ratio? Log plots covering many orders of magnitude can be hard to interpret. |
We could certainly look at dN(<m)/dmag and correct using the DEEP2 dN/dz estimates... then differentiate to get dN/dmag... it's better than nothing, at least. |
Done in #47. |
See LSSTDESC/DC2-production#21 for detail and validation data from HSC.
The text was updated successfully, but these errors were encountered: