Skip to content

SDV 2.0: Make the I/O consistent for all modalities - #2966

Draft
R-Palazzo wants to merge 7 commits into
v2.0.0from
issue-2950-i/o_consistent
Draft

SDV 2.0: Make the I/O consistent for all modalities#2966
R-Palazzo wants to merge 7 commits into
v2.0.0from
issue-2950-i/o_consistent

Conversation

@R-Palazzo

@R-Palazzo R-Palazzo commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Resolve #2950
86bbfn9gc

It’s not fully ready for review yet, but I opened a draft PR so that anyone blocked by this issue can start working from it. Some integration tests are still failing. I should have it finalized later today or tomorrow.

@R-Palazzo R-Palazzo self-assigned this Aug 27, 2026
@codecov

codecov Bot commented Aug 27, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 96.15385% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.80%. Comparing base (107a42a) to head (f05e3d1).

Files with missing lines Patch % Lines
sdv/single_table/base.py 91.52% 5 Missing ⚠️
sdv/multi_table/base.py 80.00% 1 Missing ⚠️
sdv/sequential/par.py 97.77% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           v2.0.0    #2966      +/-   ##
==========================================
+ Coverage   96.77%   96.80%   +0.03%     
==========================================
  Files          67       65       -2     
  Lines        7567     7577      +10     
==========================================
+ Hits         7323     7335      +12     
+ Misses        244      242       -2     
Flag Coverage Δ
unit 96.80% <96.15%> (+0.03%) ⬆️

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.

@sdv-team

Copy link
Copy Markdown
Contributor

This Pull Request is not linked to an issue. To ensure our community is able to accurately track resolved issues, please link any issue that will be closed by this PR!

@R-Palazzo
R-Palazzo force-pushed the issue-2950-i/o_consistent branch from 19f87ea to f05e3d1 Compare August 27, 2026 22:31
@sdv-team

Copy link
Copy Markdown
Contributor

This Pull Request is not linked to an issue. To ensure our community is able to accurately track resolved issues, please link any issue that will be closed by this PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants