fmi-basel / gliberal-scMultipleX

Feature extraction and linking of multiplexing data.
BSD 3-Clause "New" or "Revised" License
8 stars 0 forks source link

Add Fractal tasks for object and single-cell registration #99

Closed jluethi closed 9 months ago

jluethi commented 9 months ago

@nrepina I'm opening the PR so that when we push to the branch, Github CI reruns the automated tests :) Not to be merged yet

What this is adding

Minor changes:

Once fractal-tasks-core 0.14.0 is released:

codecov-commenter commented 9 months ago

Codecov Report

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

Comparison is base (4b46ca5) 98.91% compared to head (c3dd4af) 98.13%. Report is 13 commits behind head on main.

Files Patch % Lines
tests/fractal/test_fractal_measurement.py 96.66% 2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #99 +/- ## ========================================== - Coverage 98.91% 98.13% -0.79% ========================================== Files 10 10 Lines 461 482 +21 ========================================== + Hits 456 473 +17 - Misses 5 9 +4 ```

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

nrepina commented 9 months ago

Adding 4 new tasks:

The places where the i/o code could use a refactor is noted with # TODOs. In particular the calculate_platymatch_registration is huge and can benefit from some i/o helper functions.