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

Reinstate get_table_parameters for the multi-table synthesizers #1757

Closed
npatki opened this issue Jan 24, 2024 · 0 comments · Fixed by #1830
Closed

Reinstate get_table_parameters for the multi-table synthesizers #1757

npatki opened this issue Jan 24, 2024 · 0 comments · Fixed by #1830
Assignees
Labels
feature request Request for a new feature
Milestone

Comments

@npatki
Copy link
Contributor

npatki commented Jan 24, 2024

Problem Description

Currently, the multi-table synthesizers have a deprecated method called get_table_parameters. We should reinstate this method in order to align to the overall synthesizer representation. By "reinstate" we mean: Get rid of any deprecation notice/future warning.

The API of this function has slightly changed.

API

All multi-table synthesizers (HSA, HMA, Independent, DayZ), should have the same API.

Input: (required) table_name

Output: A dictionary with the following key/value pairs

  • synthesizer_name: The string name of the synthesizer for that table
    • For HSA and Independent, this could be GaussianCopulaSynthesizer, CTGANSynthesizer, TVAESynthesizer, or CopulaGANSynthesizer
    • For HMA this will always be GaussianCopulaSynthesizer
    • For muli-table DayZ this will be DayZSynthesizer (single table version)
  • synthesizer_parameters: The result of get_parameters for that single table synthesizer
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Request for a new feature
Projects
None yet
3 participants