OSOceanAcoustics / echodataflow

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

Adding Extensions to Support Main Pipeline #99

Closed Sohambutala closed 1 week ago

Sohambutala commented 3 months ago

These extensions are designed to trigger or support the main pipeline, aiding in data source acquisition, enabling concurrent deployments etc.

1. File Monitoring for Automated Deployments:

Purpose: To monitor file changes and trigger specific deployments.

Current Approach:

2. File Downloader for Seamless Data Transfer:

Purpose: To facilitate data transfer between two locations.

Implementation Details:

codecov-commenter commented 3 months ago

Codecov Report

Attention: Patch coverage is 7.19719% with 1586 lines in your changes missing coverage. Please review.

Project coverage is 31.68%. Comparing base (f85b12e) to head (83bb7d1). Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...chodataflow/stages/subflows/initialization_flow.py 9.28% 215 Missing :warning:
echodataflow/extensions/file_monitor.py 0.00% 193 Missing :warning:
echodataflow/stages/subflows/echoshader_flow.py 0.00% 144 Missing :warning:
...low/stages/subflows/echoshader_flow_predictions.py 0.00% 141 Missing :warning:
echodataflow/stages/subflows/mask_prediction.py 0.00% 120 Missing :warning:
echodataflow/extensions/file_downloader.py 0.00% 99 Missing :warning:
echodataflow/extensions/panel_app_lasker.py 0.00% 98 Missing :warning:
echodataflow/extensions/panel_app_shimada.py 0.00% 97 Missing :warning:
echodataflow/extensions/panel_app.py 0.00% 80 Missing :warning:
echodataflow/stages/subflows/Sv_pipeline.py 0.00% 80 Missing :warning:
... and 20 more

:exclamation: There is a different number of reports uploaded between BASE (f85b12e) and HEAD (83bb7d1). Click for more details.

HEAD has 1 upload less than BASE | Flag | BASE (f85b12e) | HEAD (83bb7d1) | |------|------|------| ||2|1|
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #99 +/- ## =========================================== - Coverage 50.50% 31.68% -18.82% =========================================== Files 34 48 +14 Lines 2285 3888 +1603 =========================================== + Hits 1154 1232 +78 - Misses 1131 2656 +1525 ```

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

Sohambutala commented 3 months ago

Converting this PR to draft since Prefect variables have a limit of 255 characters. An alternative would be to try using the Echodataflow block object.

valentina-s commented 2 months ago

This PR has some files that seems from an older version. I point to one example in the comment above, but I also notice some of the actions versions got reversed to the older ones. So maybe you check this, before merging.

Sohambutala commented 2 months ago

Thank you for highlighting workflow changes, I had those locally but never committed, since I used a different branch to merge into main after the hack with Caesar and Brandyn.