-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
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
Retain views with listlike indexers setitem #38204
Conversation
@jbrockmendel #38148 has been reverted, so can you update this to close #37954 and add the release note in 1.2.0 instead. |
lgtm. the perf enhancements hold here (I think so, just checking).? |
i timed it at slightly faster than the copying version (likely within the margin of error, dont recall) |
great. ping on green (not sure what's going on with CI). |
AFAICT its just backed up. A lot got merged in a short time! |
@jreback green |
thanks @jbrockmendel I think we going to backport this @simonjayhawkins (not sure if the regression was on 1.1.5)? |
no evidence this was a regression. maybe can do some digging post 1.2.0rc0 and maybe backport to 1.1.5 and move release note before 1.2.0 but for now, I would say this is not being backported. |
xref #38148