Skip to content

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