OSOceanAcoustics / echodataflow

Orchestrated sonar data processing workflow
https://echodataflow.readthedocs.io/en/latest/
MIT License
4 stars 1 forks source link

Upgrades to Deploy Automated Ship Pipeline: New Flows, Improved Logging, and Enhanced Cleanup #105

Closed Sohambutala closed 3 months ago

Sohambutala commented 4 months ago

Major Upgrades to Echodataflow Pipeline

This PR introduces significant enhancements to the echodataflow pipeline, providing new capabilities and improving existing functionalities.

Highlights:

New Flows Added

Issue Resolution:

Support for New Input:

External Parameters Support:

Added support for external parameters in all flows, enabling more customizable and dynamic workflow configurations. External parameters must match the underlying process function, and echodataflow handles them only if explicitly required for pipeline processing.

Revamped Logging:

Logging improvements include better tracking, debugging, and overall transparency. Dask logs now maintain order, log tracebacks before failures, and resolve log contamination across pipeline runs in a Dask cluster setup (resolves #102).

Improved Post-Execution Cleanup:

Enhanced cleanup logic to manage and clean up resources after execution, improving reliability and efficiency.

Exception Handling:

Added support for internal and external exceptions, clearly distinguishing between soft and hard failures.

Resolves #100

codecov-commenter commented 4 months ago

Codecov Report

Attention: Patch coverage is 23.30275% with 418 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@9979b18). Learn more about missing BASE report. Report is 18 commits behind head on main.

Files Patch % Lines
echodataflow/stages/subflows/mask_prediction.py 0.00% 77 Missing :warning:
echodataflow/stages/subflows/slice_store.py 0.00% 61 Missing :warning:
echodataflow/stages/subflows/resample.py 0.00% 60 Missing :warning:
echodataflow/stages/subflows/compute_NASC.py 0.00% 52 Missing :warning:
echodataflow/stages/subflows/write_output.py 0.00% 48 Missing :warning:
...chodataflow/stages/subflows/initialization_flow.py 48.00% 39 Missing :warning:
echodataflow/utils/file_utils.py 50.00% 16 Missing :warning:
echodataflow/stages/echodataflow.py 28.57% 15 Missing :warning:
echodataflow/stages/echodataflow_trigger.py 62.50% 15 Missing :warning:
echodataflow/stages/subflows/add_location.py 0.00% 9 Missing :warning:
... and 10 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #105 +/- ## ======================================= Coverage ? 50.15% ======================================= Files ? 34 Lines ? 2323 Branches ? 0 ======================================= Hits ? 1165 Misses ? 1158 Partials ? 0 ```

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