Skip to content

[SYSTEMDS-3977] Split Aware Execution - #2620

Merged
christinadionysio merged 16 commits into
apache:mainfrom
christinadionysio:split_aware_reps
Oct 12, 2026
Merged

christinadionysio merged 16 commits into
apache:mainfrom
christinadionysio:split_aware_reps

Conversation

@christinadionysio

@christinadionysio christinadionysio commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

This patch adds split aware execution of trained representations to avoid leaking data between train/val/test splits.

Assisted-by: AI

@codecov

codecov Bot commented Oct 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 55.81916% with 987 lines in your changes missing coverage. Please review.
✅ Project coverage is 74.27%. Comparing base (cbb62dd) to head (d7d62f1).

Files with missing lines Patch % Lines
.../python/systemds/scuro/representations/openface.py 8.60% 138 Missing ⚠️
...n/systemds/scuro/representations/video_batching.py 26.62% 124 Missing ⚠️
...main/python/systemds/scuro/representations/lstm.py 11.96% 103 Missing ⚠️
...systemds/scuro/drsearch/multimodal_ga_optimizer.py 61.58% 63 Missing ⚠️
...n/systemds/scuro/drsearch/split_aware_evaluator.py 73.66% 59 Missing ⚠️
...ain/python/systemds/scuro/models/training_stats.py 39.13% 56 Missing ⚠️
...uro/representations/multimodal_attention_fusion.py 14.03% 49 Missing ⚠️
...in/python/systemds/scuro/representations/fusion.py 44.18% 48 Missing ⚠️
src/main/python/systemds/scuro/drsearch/task.py 62.03% 41 Missing ⚠️
...on/systemds/scuro/drsearch/hyperparameter_tuner.py 68.69% 36 Missing ⚠️
... and 31 more
Additional details and impacted files
@@              Coverage Diff              @@
##               main    #2620       +/-   ##
=============================================
+ Coverage     71.89%   74.27%    +2.37%     
=============================================
  Files          2081      437     -1644     
  Lines        223652    26568   -197084     
  Branches      38581        0    -38581     
=============================================
- Hits         160797    19733   -141064     
+ Misses        51706     6835    -44871     
+ Partials      11149        0    -11149     
Flag Coverage Δ
java ?
python 74.27% <55.81%> (-0.68%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@christinadionysio
christinadionysio merged commit 80817a9 into apache:main Oct 12, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant