LSSTDESC / rail_base

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

WIP, Lots of delinting with pylint #79

Closed eacharles closed 4 months ago

eacharles commented 9 months ago

Problem & Solution Description (including issue #)

For now this is just a draft to run the unit tests on this branch. Once some of the other changes get merged we can talk about merging this. Although there are a lot of changes, this should have no effect on running the code.

Code Quality

codecov[bot] commented 9 months ago

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (fd13285) 97.17% compared to head (28323f5) 98.15%. Report is 5 commits behind head on main.

Files Patch % Lines
src/rail/estimation/summarizer.py 56.52% 10 Missing :warning:
src/rail/core/data.py 97.50% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #79 +/- ## ========================================== + Coverage 97.17% 98.15% +0.98% ========================================== Files 32 32 Lines 1736 1683 -53 ========================================== - Hits 1687 1652 -35 + Misses 49 31 -18 ```

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

eacharles commented 4 months ago

This has been merged into the eac-evaluation branch and will be merged what that is merged. closing the PR.