Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add REalTabFormer to supported synthesizers #360

Merged
merged 37 commits into from
Dec 12, 2024

Conversation

cristid9
Copy link
Contributor

@cristid9 cristid9 commented Nov 5, 2024

Resolves #347
CU-86b2czk01

@cristid9 cristid9 requested a review from a team as a code owner November 5, 2024 15:55
@cristid9 cristid9 requested review from amontanez24 and removed request for a team November 5, 2024 15:55
@sdv-team
Copy link
Contributor

sdv-team commented Nov 5, 2024

@pvk-developer pvk-developer self-requested a review November 5, 2024 16:01
@cristid9 cristid9 force-pushed the realtabformer_custom_synthesizer branch from 4b6b947 to 58bd0d4 Compare November 5, 2024 19:13
Copy link
Contributor

@amontanez24 amontanez24 left a comment

Choose a reason for hiding this comment

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

Let's add some unit tests

tests/integration/test_benchmark.py Outdated Show resolved Hide resolved
tests/integration/test_benchmark.py Show resolved Hide resolved
tests/integration/test_benchmark.py Outdated Show resolved Hide resolved
tests/unit/synthesizers/test_realtabformer.py Outdated Show resolved Hide resolved
tests/unit/synthesizers/test_realtabformer.py Outdated Show resolved Hide resolved
tests/unit/synthesizers/test_realtabformer.py Outdated Show resolved Hide resolved
sdgym/synthesizers/realtabformer.py Outdated Show resolved Hide resolved
@cristid9 cristid9 force-pushed the realtabformer_custom_synthesizer branch 9 times, most recently from 12a1495 to 43d446e Compare November 19, 2024 12:22
tests/integration/test_benchmark.py Outdated Show resolved Hide resolved
tests/integration/test_benchmark.py Outdated Show resolved Hide resolved
tests/unit/synthesizers/test_realtabformer.py Outdated Show resolved Hide resolved
@cristid9 cristid9 force-pushed the realtabformer_custom_synthesizer branch 4 times, most recently from a64d60f to 5593604 Compare November 27, 2024 15:57
@cristid9 cristid9 force-pushed the realtabformer_custom_synthesizer branch 2 times, most recently from 1b279c9 to b679a3c Compare November 27, 2024 16:45
@cristid9 cristid9 force-pushed the realtabformer_custom_synthesizer branch from 0ab4148 to b84a641 Compare December 9, 2024 22:13
.github/workflows/integration.yml Outdated Show resolved Hide resolved
.github/workflows/minimum.yml Outdated Show resolved Hide resolved
.github/workflows/integration.yml Outdated Show resolved Hide resolved
Comment on lines 91 to 92
assert train_time >= 0
assert sample_time >= 0
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is the sample time 0 for windows? Is it crashing or rounding incorrectly?

tests/integration/test_benchmark.py Outdated Show resolved Hide resolved
tests/integration/test_benchmark.py Outdated Show resolved Hide resolved
Copy link
Member

@pvk-developer pvk-developer left a comment

Choose a reason for hiding this comment

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

After addressing my feedback this should be good to go!

.github/workflows/minimum.yml Outdated Show resolved Hide resolved
.github/workflows/unit.yml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@cristid9 cristid9 merged commit 50ac89b into main Dec 12, 2024
47 checks passed
@cristid9 cristid9 deleted the realtabformer_custom_synthesizer branch December 12, 2024 12:49
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.

Add integration with 3rd party synthesizer (REalTabFormer)
4 participants