FarrOutLab / GWInferno

GWInferno: Gravitational-Wave Hierarchical Inference with NumPyro
MIT License
14 stars 5 forks source link

Reorganize #104

Closed jaxengodfrey closed 4 months ago

jaxengodfrey commented 4 months ago

Changes:

gwinferno.models.gwpopulation.gwpopulation --> gwinferno.models.parametric.parametric (closes #99) gwinferno.models.mass_gap --> deleted ❌ (closes #100) gwinferno.types --> deleted ❌ (closes #98) gwinferno.postprocess.calculate_ppds --> deletedgwinferno.postprocess.plotting --> deletedgwinferno.types --> deleted❌ (closes #71)

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 48.93%. Comparing base (1354f80) to head (d77d6c8).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #104 +/- ## =========================================== + Coverage 37.02% 48.93% +11.90% =========================================== Files 33 27 -6 Lines 2490 1825 -665 =========================================== - Hits 922 893 -29 + Misses 1568 932 -636 ``` | [Flag](https://app.codecov.io/gh/FarrOutLab/GWInferno/pull/104/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FarrOutLab) | Coverage Δ | | |---|---|---| | [python3.10](https://app.codecov.io/gh/FarrOutLab/GWInferno/pull/104/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FarrOutLab) | `48.93% <100.00%> (+11.90%)` | :arrow_up: | | [python3.11](https://app.codecov.io/gh/FarrOutLab/GWInferno/pull/104/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FarrOutLab) | `48.93% <100.00%> (?)` | | | [python3.9](https://app.codecov.io/gh/FarrOutLab/GWInferno/pull/104/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FarrOutLab) | `48.81% <100.00%> (+11.91%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=FarrOutLab#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

jaxengodfrey commented 4 months ago

Created new branch development then removed 2D B-Spline functions on current branch (these will remain in development branch): (closes #92)

gwinferno.models.bsplines.joint --> deletedgwinferno.models.bpslines.smoothing.adaptive_difference_prior --> deletedgwinferno.models.bpslines.smoothing.adaptive_twod_difference_prior --> deletedgwinferno.models.bpslines.smoothing.get_adaptive_lambda --> deletedgwinferno.models.bpslines.smoothing.mixture_smoothing_parameter --> deleted