-
Notifications
You must be signed in to change notification settings - Fork 443
Update class_weights to support list[float] #2707
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
base: main
Are you sure you want to change the base?
Conversation
Could use https://docs.python.org/3/library/warnings.html#warnings.deprecated or explicitly raise https://docs.python.org/3/library/exceptions.html#DeprecationWarning for deprecation. I'm not sure why the doc tests are failing either, but you're not the only PR with failing tests. Will try to investigate this later if it keeps happening. |
Restarting the docs build fixed the other PR, hoping this is just a transient issue. |
Hmm did this change break this test?
|
Nope, this is because I used random numbers which are sometimes outside the valid range. Will submit a fix. If you rerun it should only fail 1% of the time. |
Is there a |
Yes, just hit it. I don't think you have access, but I'm happy to rerun until CI is happy. |
If you could - I dont see the button 🥹 |
@adamjstewart thoughts? |
Let's discuss more on #2060 |
Close #2060
Could be cleaner to deprecate tensor support, not sure if there is a process to this?
Validated with