Closed ismael-mendoza closed 5 months ago
Check out this pull request on
See visual diffs & provide feedback on Jupyter Notebooks.
Powered by ReviewNB
Attention: Patch coverage is 24.07407%
with 41 lines
in your changes are missing coverage. Please review.
Project coverage is 62.76%. Comparing base (
180bf61
) to head (cead309
).:exclamation: Current head cead309 differs from pull request most recent head 655ce30. Consider uploading reports for the commit 655ce30 to get more accurate results
Files | Patch % | Lines |
---|---|---|
btk/match.py | 33.33% | 22 Missing :warning: |
btk/measure.py | 13.33% | 13 Missing :warning: |
btk/deblend.py | 0.00% | 5 Missing :warning: |
btk/plotting.py | 0.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This PR will adjust the following two notebooks so they are ready to be released alongside the BTK paper:
quickstart
advanced-analysis
In
advanced-analysis
, I'm also tidying up the plots to be "paper-ready". Other todo items:advanced-analysis
or understand why they happen.In the end, when looking into the above to-do item I discovered some bugs in the matching procedure. So I'm fixing them here too.