-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
DEPR: Index.insert dtype-inference #55257
Conversation
Its kind of annoying that this warning gets issued (the PR currently suppress it) in |
This pull request is stale because it has been open for thirty days with no activity. Please update and respond to this comment if you're still interested in working on this. |
@phofl thoughts on doing this? i think there was some vague similarity to some of what you were working on. main open question here is whether to continue suppressing warnings on setitem-with-expansion |
thx @jbrockmendel |
Glad to get movement here, but i was hoping for some discussion of whether suppressing the warning in setitem-with-expansion was the way to go |
I wouldn’t want to warn there, I don’t think that anyone cares about inference there and getting rid of it is a pain |
doc/source/whatsnew/vX.X.X.rst
file if fixing a bug or adding a new feature.Doesn't have a "do X to avoid this warning, not sure if this merits future option or temporary keyword.