UCLH-Foundry / PIXL

PIXL Image eXtraction Laboratory
Apache License 2.0
8 stars 0 forks source link

Process MRIs on GAE #378

Closed stefpiatek closed 2 months ago

stefpiatek commented 2 months ago

Tags uses for MS-PINPOINT and prostate MRI projects.

fixes #368

Also some fixes during running:

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 95.65217% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 80.61%. Comparing base (fc8e962) to head (7d004d0).

Files Patch % Lines
pixl_core/src/core/uploader/base.py 91.66% 1 Missing :warning:
...l_core/tests/project_config/test_project_config.py 85.71% 1 Missing :warning:
pixl_export/src/pixl_export/main.py 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #378 +/- ## ========================================== - Coverage 82.97% 80.61% -2.36% ========================================== Files 79 76 -3 Lines 3259 3085 -174 ========================================== - Hits 2704 2487 -217 - Misses 555 598 +43 ```

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

stefpiatek commented 2 months ago

Looks good; nothing here is intended to hold up merging if you need to get it in today. I'm not able to give an opinion on the specific tags referenced here.

Thanks! I'll wait for a post-merge review from @HChughtai when he's back before we set off the final export for ms-pinpoint