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

[SUGGESTION] - Replace comments to clear the confusion #283

Open
majidliaquat opened this issue Jun 22, 2023 · 1 comment
Open

[SUGGESTION] - Replace comments to clear the confusion #283

majidliaquat opened this issue Jun 22, 2023 · 1 comment
Labels
documentation Improvements or additions to documentation

Comments

@majidliaquat
Copy link

HI, I new to Ydata-Synthetic, I just want to highlight that while observing the visualisation of PCA and TSNE in Google Colab example file of Time Series synthetic data generation with TimeGAN on stock dataset I was confuse first reading the comments in code.

I know it may not be difficult for someone by looking into code but comments are also important to be on best line.
I am attaching the screenshot, also if there is a confusion from myside that I am not getting it correctly please let me know.

Thanks
Screenshot 2023-06-22 at 11 17 38 AM

@majidliaquat majidliaquat added the bug Something isn't working label Jun 22, 2023
@fabclmnt
Copy link
Contributor

Hi @majidliaquat,

feel free to contribute with the changes. Help with documentation is always welcome.

I'l going to remove the bug label as this is rather a docs update.

@fabclmnt fabclmnt added documentation Improvements or additions to documentation and removed bug Something isn't working labels Jul 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants