Skip to content

Backport PR #47550 on branch 1.4.x (REGR: fix eval with inplace=True to correctly update column values inplace) #48151

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

Merged
merged 2 commits into from
Aug 19, 2022

Conversation

phofl
Copy link
Member

@phofl phofl commented Aug 19, 2022

…place (pandas-dev#47550)

* fix pre-commit issues

* fix linting errors

* add inplace argument to isetitem and use in eval

* changes due to PR discussions

* fix issues

* update eval

* update whatsnew

* add PR suggestions

* update imports in eval.py

* check inplace and use NDFrame + small update to test

* update test to use using_copy_on_write

* skip test for array manager

Co-authored-by: Joris Van den Bossche <[email protected]>
@phofl phofl added this to the 1.4.4 milestone Aug 19, 2022
@phofl phofl added Regression Functionality that used to work in a prior pandas version expressions pd.eval, query labels Aug 19, 2022
@phofl phofl changed the title Backport PR #47550 on branch 1.4.x (CI/DEPS: Fix timezone test due to pytz upgrade) Backport PR #47550 on branch 1.4.x (REGR: fix eval with inplace=True to correctly update column values inplace) Aug 19, 2022
Copy link
Member

@simonjayhawkins simonjayhawkins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @phofl merge on green(ish)

@phofl phofl merged commit 233ebdf into pandas-dev:1.4.x Aug 19, 2022
@phofl phofl deleted the backport_eval branch August 19, 2022 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
expressions pd.eval, query Regression Functionality that used to work in a prior pandas version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants