-
Notifications
You must be signed in to change notification settings - Fork 37
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
Simple metrics for density uniformity #397
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some questions about the FskyMetric.
remove_monopole looks good.
I'll try to have a look at the notebook tomorrow, and see if I can say anything helpful! |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #397 +/- ##
==========================================
+ Coverage 55.25% 60.16% +4.90%
==========================================
Files 305 296 -9
Lines 29681 28278 -1403
Branches 4285 4022 -263
==========================================
+ Hits 16400 17013 +613
+ Misses 12338 10267 -2071
- Partials 943 998 +55 ☔ View full report in Codecov by Sentry. |
Some nitpicks feedback (docstrings, line length, black, all's, breaking things into new files) implemented in a PR on the fork - ixkael#1 |
It would still be good to add test coverage. |
Reformatting and minor changes
""" | ||
Run as summary metric on NestedRIZExptimeExgalM5Metric. | ||
|
||
This metric StaticProbesFoMEmulatorMetric |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this needs to be expanded? Something like:
This metric uses maps of the combined RIZ exposure time and i-band depth maps (with a consistent set of area cuts) to identify potential reductions in cosmological constraining power due to substantial large-scale power in non-uniform coadds at a particular data release. The RIZ exposure time map is used to identify whether there are residual rolling features. If not, the metric returns 1. If there are such features, then the region is segmented into similar-depth regions and the one with the largest cosmological constraining power is presumed to be used for science. In that case, the metric returns the 3x2pt FoM (StaticProbesFoMEmulatorMetric
, quantifying weak lensing and large-scale structure constraining power) for the largest of those regions, divided by that for the full region if it had been usable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for catching this gap Rachel! I committed and push this change.
Sync with my fork of Boris' rubin_sim
This PR has three simple new metrics:
They come together in a sigma8 tomography metric which is demonstrated here: https://github.com/ixkael/ObsStrat/blob/meanz_uniformity_maf/code/meanz_uniformity/sigma8tomography_demo.ipynb
Annoyingly I didn't have time to figure out how to properly code this in a nice metric. This is partly because I couldn't think of an elegant way to avoid multiple calls to other existing metrics (including the new LinearMultibandModelMetric). For example the average depth is needed, and I wasn't sure how to avoid calling the ExgalM5 twice. The other problem is the need to use the depth maps N times (with N the number of redshift bins considered, here 5). Again I couldn't think of a nice way to run the metrics in a nested way in time for this deadline. But I hope the demo notebook illustrates things clearly enough, and I am happy to assist in any way I can.