LSSTDESC / rail_base

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

change len in trainz from hardcoded value #32

Closed sschmidt23 closed 1 year ago

sschmidt23 commented 1 year ago

This is a simple one-line change where there was a hardcoded len('mag_i_lsst') to determine the number of galaxies to make duplicate trainZ distributions for. Now we just do end-start, which in process chunk gives us that same data length.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (91adfec) 97.53% compared to head (093c5df) 97.53%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #32 +/- ## ======================================= Coverage 97.53% 97.53% ======================================= Files 28 28 Lines 1585 1585 ======================================= Hits 1546 1546 Misses 39 39 ``` | [Files Changed](https://app.codecov.io/gh/LSSTDESC/rail_base/pull/32?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC) | Coverage Δ | | |---|---|---| | [src/rail/estimation/algos/train\_z.py](https://app.codecov.io/gh/LSSTDESC/rail_base/pull/32?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-c3JjL3JhaWwvZXN0aW1hdGlvbi9hbGdvcy90cmFpbl96LnB5) | `100.00% <100.00%> (ø)` | |

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