i4Ds / Karabo-Pipeline

The Karabo Pipeline can be used as Digital Twin for SKA
https://i4ds.github.io/Karabo-Pipeline/
MIT License
11 stars 4 forks source link

Added transpose to get_pixel_position_of_sources() #492

Closed kenfus closed 1 year ago

kenfus commented 1 year ago
codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -1.02% :warning:

Comparison is base (adfe033) 68.67% compared to head (e8e0277) 67.66%. Report is 6 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #492 +/- ## ========================================== - Coverage 68.67% 67.66% -1.02% ========================================== Files 34 34 Lines 3761 3761 Branches 475 475 ========================================== - Hits 2583 2545 -38 - Misses 1001 1041 +40 + Partials 177 175 -2 ``` | [Files Changed](https://app.codecov.io/gh/i4Ds/Karabo-Pipeline/pull/492?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=i4Ds) | Coverage Δ | | |---|---|---| | [karabo/sourcedetection/result.py](https://app.codecov.io/gh/i4Ds/Karabo-Pipeline/pull/492?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=i4Ds#diff-a2FyYWJvL3NvdXJjZWRldGVjdGlvbi9yZXN1bHQucHk=) | `59.32% <100.00%> (-14.41%)` | :arrow_down: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/i4Ds/Karabo-Pipeline/pull/492/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=i4Ds)

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

Lukas113 commented 1 year ago

In addition, I believe codecov/project failed because there is not coverage-report at the latest commit on the main branch. But because the "error message" in Details doesn't show any meaningful information why it fails I'm not sure about that. I also haven't found something useful after a google-search.

So I just hope that after #491 is merged successfully it will work. Otherwise I must have a look at this issue.