Skip to content

Add TabDDPM to SDGym single table synthesizers#628

Open
sarahmish wants to merge 4 commits into
mainfrom
tabddpm
Open

Add TabDDPM to SDGym single table synthesizers#628
sarahmish wants to merge 4 commits into
mainfrom
tabddpm

Conversation

@sarahmish

Copy link
Copy Markdown
Contributor

@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.74311% with 50 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.89%. Comparing base (3b2afff) to head (431e053).

Files with missing lines Patch % Lines
sdgym/synthesizers/tabddpm.py 92.73% 50 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #628      +/-   ##
==========================================
+ Coverage   85.81%   86.89%   +1.08%     
==========================================
  Files          40       41       +1     
  Lines        3722     4411     +689     
==========================================
+ Hits         3194     3833     +639     
- Misses        528      578      +50     
Flag Coverage Δ
integration 51.68% <92.30%> (+7.51%) ⬆️
unit 83.24% <92.16%> (+1.65%) ⬆️

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.

@sarahmish
sarahmish marked this pull request as ready for review July 17, 2026 15:13
@sarahmish
sarahmish requested a review from a team as a code owner July 17, 2026 15:13
@sarahmish
sarahmish requested review from R-Palazzo and amontanez24 and removed request for a team July 17, 2026 15:13
@R-Palazzo
R-Palazzo changed the base branch from main to stable July 17, 2026 15:18
@R-Palazzo
R-Palazzo changed the base branch from stable to main July 17, 2026 15:18
@sarahmish sarahmish self-assigned this Jul 17, 2026

@R-Palazzo R-Palazzo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good!

Do we expect this PR to support running TabDDPMSynthesizer in the single-table benchmark, including running it on AWS and GCP?

If so, could we add integration tests for benchmark_single_table()?

For AWS and GCP, I previously tested this manually and shared the result folder in the PR. For example, for #549, this is the test folder

I can help for testing it on AWS and GCP

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