Skip to content
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: Series/DataFrame.transform partial failure except TypeError #40288

Merged
merged 8 commits into from
Mar 26, 2021

Conversation

rhshadrach
Copy link
Member

@rhshadrach rhshadrach commented Mar 7, 2021

Partial revert of #40238. I did not realize how TypeError was handled differently than other errors in other UDF functions (#40211 (comment)). The deprecation is now only for partial failure when something other than TypeError is raised.

In order to make this work, transform needed to passthrough TypeErrors when they occur so we can correctly decide how to act with a list- or dict-like.

@rhshadrach rhshadrach added Deprecate Functionality to remove in pandas Apply Apply, Aggregate, Transform labels Mar 7, 2021
@rhshadrach rhshadrach marked this pull request as draft March 10, 2021 00:25
@rhshadrach rhshadrach marked this pull request as ready for review March 13, 2021 17:56
…ansform_partial_failure

� Conflicts:
�	doc/source/whatsnew/v1.3.0.rst
pandas/core/apply.py Outdated Show resolved Hide resolved
@jreback jreback added this to the 1.3 milestone Mar 26, 2021
@jreback jreback merged commit dd943c8 into pandas-dev:master Mar 26, 2021
@jreback
Copy link
Contributor

jreback commented Mar 26, 2021

thanks @rhshadrach

@rhshadrach rhshadrach deleted the transform_partial_failure branch March 26, 2021 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Apply Apply, Aggregate, Transform Deprecate Functionality to remove in pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API: DataFrame.agg has no partial failure
3 participants