search

LSSTDESC / firecrown

DESC Cosmology Likelihood Framework
BSD 3-Clause "New" or "Revised" License
29 stars 7 forks source link

Fix bug 413 and friends #418

Closed marcpaterno closed 3 months ago

marcpaterno commented 3 months ago

This will close issue #413 . It also simplifies the __eq__ for TupleBin and fixes some flake8 complaints.

codecov[bot] commented 3 months ago

Codecov Report

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

Project coverage is 97.9%. Comparing base (53117a7) to head (e5e1b93). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #418 +/- ## ====================================== Coverage 97.8% 97.9% ====================================== Files 35 35 Lines 2982 2974 -8 ====================================== - Hits 2919 2912 -7 + Misses 63 62 -1 ``` | [Files](https://app.codecov.io/gh/LSSTDESC/firecrown/pull/418?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC) | Coverage Δ | | |---|---|---| | [firecrown/likelihood/gauss\_family/gauss\_family.py](https://app.codecov.io/gh/LSSTDESC/firecrown/pull/418?src=pr&el=tree&filepath=firecrown%2Flikelihood%2Fgauss_family%2Fgauss_family.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-ZmlyZWNyb3duL2xpa2VsaWhvb2QvZ2F1c3NfZmFtaWx5L2dhdXNzX2ZhbWlseS5weQ==) | `99.3% <ø> (ø)` | | | [...own/likelihood/gauss\_family/statistic/two\_point.py](https://app.codecov.io/gh/LSSTDESC/firecrown/pull/418?src=pr&el=tree&filepath=firecrown%2Flikelihood%2Fgauss_family%2Fstatistic%2Ftwo_point.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-ZmlyZWNyb3duL2xpa2VsaWhvb2QvZ2F1c3NfZmFtaWx5L3N0YXRpc3RpYy90d29fcG9pbnQucHk=) | `97.2% <100.0%> (+0.3%)` | :arrow_up: | | [firecrown/models/cluster/binning.py](https://app.codecov.io/gh/LSSTDESC/firecrown/pull/418?src=pr&el=tree&filepath=firecrown%2Fmodels%2Fcluster%2Fbinning.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-ZmlyZWNyb3duL21vZGVscy9jbHVzdGVyL2Jpbm5pbmcucHk=) | `100.0% <100.0%> (ø)` | |