-
Notifications
You must be signed in to change notification settings - Fork 27
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
a little mistake in the code #14
Comments
Thanks for taking the time to file an issue. I think you're right. Feel free to file a PR, if you like. We haven't been working on this project in a while, but I have another project that built on this balancing codebase, should you be interested in using it: https://github.com/tgsmith61591/skoot |
The original SMOTE paper got its biggest performance boost by combining down-sampling with their method. I think it would absolutely be worth trying such a method. Things like:
The skoot package I shared can help you downsample and perform SMOTE, but the other strategies will depend on your framework and family of algorithms. |
Thanks for your help. |
smrt/smrt/balance/smrt.py
Line 207 in 0863b0b
I think this should be
X_sub = X_copy[y_transform == transformed_label, :]
The text was updated successfully, but these errors were encountered: