You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
As first described in #469, it seems that whenever Copulas is asked to print parameters for a fitted GaussianKDE distribution, it just prints out a copy of the data that was fitted.
In the code below, the final column (column z) is fitted to a GaussianKDE distribution.
Ideally, the _params assigned to the GaussianKDE should be None, GaussianKDE is non-parametric distribution. Whatever info we need to save the state of the GassianKDE should be saved under a different name and not exposed as parameters.
The text was updated successfully, but these errors were encountered:
npatki
added
the
bug
There is an error in the code that needs to be fixed
label
May 14, 2025
Environment Details
Error Description
As first described in #469, it seems that whenever Copulas is asked to print parameters for a fitted GaussianKDE distribution, it just prints out a copy of the data that was fitted.
In the code below, the final column (column
z
) is fitted to a GaussianKDE distribution.The data seems to be just be the exact values in column
z
Expected Behavior
It's unexpected that the entire column's data would be reported at this step.
I would expect that when printing out the distribution, it would only show the
'type'
of distribution and nothing else.It seems like the "parameters" are set to the data in fit portion:
Copulas/copulas/univariate/gaussian_kde.py
Lines 166 to 172 in 356be32
Ideally, the
_params
assigned to the GaussianKDE should beNone
, GaussianKDE is non-parametric distribution. Whatever info we need to save the state of the GassianKDE should be saved under a different name and not exposed as parameters.The text was updated successfully, but these errors were encountered: