arjunsavel / SImMER

An open-source & modular reduction pipeline for astronomical images of point sources.
https://simmer.readthedocs.io
Other
8 stars 2 forks source link

change to logging #169

Closed arjunsavel closed 8 months ago

arjunsavel commented 8 months ago

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

Checklist:

codecov[bot] commented 8 months ago

Codecov Report

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

Comparison is base (0d8d3b5) 74.58% compared to head (612260d) 76.23%. Report is 9 commits behind head on main.

Files Patch % Lines
src/simmer/analyze_image.py 10.53% 17 Missing :warning:
src/simmer/check_logsheet.py 71.43% 4 Missing :warning:
src/simmer/image.py 50.00% 4 Missing :warning:
src/simmer/registration.py 50.00% 2 Missing :warning:
src/simmer/plotting.py 66.67% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #169 +/- ## ========================================== + Coverage 74.58% 76.23% +1.65% ========================================== Files 23 25 +2 Lines 1900 2301 +401 ========================================== + Hits 1417 1754 +337 - Misses 483 547 +64 ``` | [Flag](https://app.codecov.io/gh/arjunsavel/SImMER/pull/169/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Arjun+Savel) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/arjunsavel/SImMER/pull/169/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Arjun+Savel) | `76.23% <49.09%> (?)` | | 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=Arjun+Savel#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.