-
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
pull: Support NPZ format in current transformer #167
base: feature-npz-support
Are you sure you want to change the base?
Conversation
…nous_transform method
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #167 +/- ##
=======================================
Coverage 80.09% 80.09%
=======================================
Files 69 69
Lines 3305 3305
=======================================
Hits 2647 2647
Misses 658 658 ☔ View full report in Codecov by Sentry. |
… format where the method is called
for more information, see https://pre-commit.ci
Adding tests for DataTransformer._output_column_data method -Making sure it correctly processes 'npz' output type by calling numpy.savez with expected arguments. - Verify appropriate error handling and messaging for unsupported output types.
for more information, see https://pre-commit.ci
@sirisneupane123 @sam3179 Thank you for the PR. We'll also do some code review later. Viewing from the current code structure, demo.py should be moved to the example/ directory (and renamed), and test should be added to the existing unit tests in the tests/ directory. |
Why is there a test case in sdgx/models/components/sdv_ctgan/data_transformer_test.py and it's using unittest's style Is there any reason not to put it under tests, as well as not using pytest? |
…rator into Craig # Conflicts: # demo.py
…onstrate_issue_33() to demonstrate solved issue 33
for more information, see https://pre-commit.ci
…ping in the debugging of issues related to incorrect model versions being used or misconfigured paths.
… from a file or directly passed as an object. It's particularly useful when errors may be associated with metadata formatting or source, or maybe in complex systems where multiple metadata configurations might exist.
for more information, see https://pre-commit.ci
…synthesizer with metadata and data processors
for more information, see https://pre-commit.ci
FAQ: by Siris Neupane
FAQ by Siris Neupane
for more information, see https://pre-commit.ci
…to a metadata object to a json file
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
…nstead of production file
# Conflicts: # sdgx/data_models/metadata.py
for more information, see https://pre-commit.ci
… type and object column type
for more information, see https://pre-commit.ci
Description
Motivation and Context
How has this been tested?
Types of changes
Checklist: