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

Investigate convergence warnings & silence if they can be safely ignored #202

Open
sparshgarg23 opened this issue Jan 11, 2022 · 1 comment
Labels
feature request Request for a new feature

Comments

@sparshgarg23
Copy link

Environment Details

Please indicate the following details about the environment in which you found the bug:

  • CTGAN version:0.5.0
  • Python version:3.9.7
  • Operating System:windows 10

Error Description

While running CTGAN demo on my system,I encountered the following warning
ConvergenceWarning: Initialization 1 did not converge. Try different init parameters, or increase max_iter, tol or check for degenerate data.
warnings.warn('Initialization %d did not converge.

Steps to reproduce

Use pip install ctgan
Execute demo code on windows operaitng system

python ctgan_demo.py
@sparshgarg23 sparshgarg23 added bug Something isn't working pending review This issue needs to be further reviewed, so work cannot be started labels Jan 11, 2022
@npatki
Copy link
Contributor

npatki commented Jul 12, 2022

Hi @sparshgarg23, the convergence warnings may indicate that the model having some trouble with mathematical optimization for that particular epoch. I see this frequently. It is not indicative of a bug and it usually does not correspond to any degradation in synthetic data quality.

Let's repurpose this issue as a feature request to investigate such warnings and figure out if we can safely silence them so they are not alarming during the fitting phase.

@npatki npatki added feature request Request for a new feature and removed bug Something isn't working pending review This issue needs to be further reviewed, so work cannot be started labels Jul 12, 2022
@npatki npatki changed the title convergence warning while running CTGAN demo Investigate convergence warnings & silence if they can be safely ignored Jul 12, 2022
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

No branches or pull requests

2 participants