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.

Sign up for GitHub

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

Jump to bottom

Modularize CTGAN fit function #139

Open
fealho opened this issue Mar 12, 2021 · 0 comments
Open

Modularize CTGAN fit function #139

fealho opened this issue Mar 12, 2021 · 0 comments
Labels
internal The issue doesn't change the API or functionality

Comments

@fealho
Copy link
Member

fealho commented Mar 12, 2021 •
edited
Loading

Currently, the fit function for the CTGAN synthesizer is a 140 line piece of dense code accomplishing a lot of important tasks. Modularizing it in to the same standard as some of our other libraries (such as DeepEcho) would make it much more readable and testable.
@fealho fealho added the internal The issue doesn't change the API or functionality label Mar 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
internal The issue doesn't change the API or functionality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@fealho