-
Notifications
You must be signed in to change notification settings - Fork 541
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
[0.1.0]Breaking changes: Reactoring models Part 1 #68
Conversation
Apply Sweep Rules to your PR?
|
Due to sdv-dev changed their license to BSL License, we copy the MIT licensed code into |
This is the first part of the refactoring model, mainly for the preparation of the replacement of CTGAN, which involves In the second part, I will replace the current After this, I hope we can port the performance optimisations and compatibility modifications that we did before. A series of documentation updates and development of CLI tools will also be needed. |
7624f25
to
68a83b0
Compare
For the sake of review, I've split the PR into two parts, the first being the Inspector and Metadata changes, and the other being the BSL Licence changes. @MooooCat If there are no problems, I will first merge the second part into the |
Cool, some algorithm-related branches in development may also synchronize the latest components. |
* Cleaning dep * Dropping BSL License package * Fixing example(for now)
Description
Motivation and Context
DiscreteInspector
forCTGAN
, which useful for guessing discrete columnsMetadata
support(vanilla version for now)How has this been tested?
Types of changes
Checklist: