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

The GaussianNormalizer should accept uniform distributions #655

Closed
npatki opened this issue Jun 22, 2023 · 0 comments · Fixed by #706
Closed

The GaussianNormalizer should accept uniform distributions #655

npatki opened this issue Jun 22, 2023 · 0 comments · Fixed by #706
Assignees
Labels
feature request Request for a new feature
Milestone

Comments

@npatki
Copy link
Contributor

npatki commented Jun 22, 2023

Problem Description

The GaussianNormalizer accepts a distribution parameter, where you can provide a string name of the univariate distribution. For example:

my_transformer = GaussianNormalizer(distribution='beta')

The available distributions are: 'gaussian', 'gamma', 'beta', 'student_t', 'truncated_gaussian' and 'gaussian_kde'

Expected behavior

We should also accept 'uniform' as a possible parameter. When selected, this should use the copulas UniformUnivariate distribution.

Additional context

If you're only using RDT, then instead of supplying the distribution as a string name, you can also directly input a class. This also does not work for uniform distributions.

from copulas import univariate

my_transformer = GaussianNormalizer(distribution=univariate.Uniform)
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
Development

Successfully merging a pull request may close this issue.

3 participants