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.

Sign up for GitHub

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

Jump to bottom

More splits than classes #46

Closed
fealho opened this issue Jan 27, 2021 · 0 comments · Fixed by #47
Closed

More splits than classes #46

fealho opened this issue Jan 27, 2021 · 0 comments · Fixed by #47
Assignees
@fealho
Labels
bug Something isn't working
Milestone
0.1.2

Comments

@fealho
Copy link
Member

fealho commented Jan 27, 2021

The code crashes when there are less member of a class than the requested number of splits for the dataset. Having a try/except inside single table detection metrics base and returning Nan could solve the issue.

ValueError: n_splits=3 cannot be greater than the number of members in each class.
@fealho fealho added the bug Something isn't working label Jan 27, 2021
@fealho fealho mentioned this issue Jan 27, 2021
Merged
@csala csala closed this as completed in #47 Jan 27, 2021
@csala csala added this to the 0.1.2 milestone Jan 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fealho fealho

Labels
bug Something isn't working
Projects
None yet
Milestone
0.1.2
Development

Successfully merging a pull request may close this issue.

Fixes more splits than classes issue sdv-dev/SDMetrics
2 participants
@csala @fealho