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
@github-project-automation github-project-automation Bot moved this from In Progress to Done in SystemDS PR Queue Oct 12, 2026
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