-
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
Breaking changes: Refactoring of new architecture #56
Conversation
Apply Sweep Rules to your PR?
|
I'll ping reviewers when it's ready. Feel free to comment anytime, thanks! |
374e376
to
a8d6d03
Compare
1dfb441
to
0c92137
Compare
This PR is long enough with the following modifications:
I'd like to request to merge this PR first for others who can benefit from the framework.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad to see more further results achieved under the new architecture.
Description
I will be designing the overall project architecture for the betterment of the project.
Until this PR is merged, all design and related documentation changes will be placed in here: https://synthetic-data-generator.readthedocs.io/en/breaking-refactoring/index.html
Motivation and Context
Motivation: https://synthetic-data-generator.readthedocs.io/en/breaking-refactoring/design/motivation.html
Architecture: https://synthetic-data-generator.readthedocs.io/en/breaking-refactoring/design/architecture.html
How has this been tested?
Types of changes
Checklist: