LSSTDESC / rail_base

Base classes for RAIL
MIT License
0 stars 1 forks source link

move finalize from run() to __call__() to match other Stages #71

Closed eacharles closed 9 months ago

eacharles commented 9 months ago

Problem & Solution Description (including issue #)

Code Quality

codecov[bot] commented 9 months ago

Codecov Report

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

Comparison is base (b886bc6) 96.96% compared to head (7ad59b2) 96.96%. Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #71 +/- ## ======================================= Coverage 96.96% 96.96% ======================================= Files 32 32 Lines 1712 1714 +2 ======================================= + Hits 1660 1662 +2 Misses 52 52 ``` | [Files](https://app.codecov.io/gh/LSSTDESC/rail_base/pull/71?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC) | Coverage Δ | | |---|---|---| | [src/rail/core/data.py](https://app.codecov.io/gh/LSSTDESC/rail_base/pull/71?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-c3JjL3JhaWwvY29yZS9kYXRhLnB5) | `99.23% <100.00%> (ø)` | | | [src/rail/core/util\_stages.py](https://app.codecov.io/gh/LSSTDESC/rail_base/pull/71?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-c3JjL3JhaWwvY29yZS91dGlsX3N0YWdlcy5weQ==) | `100.00% <100.00%> (ø)` | | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/LSSTDESC/rail_base/pull/71/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC)

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