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

REGR: df.apply('any', axis=1) fails with ValueError #48656

Closed
3 tasks done
vnlitvinov opened this issue Sep 20, 2022 · 3 comments · Fixed by #48797
Closed
3 tasks done

REGR: df.apply('any', axis=1) fails with ValueError #48656

vnlitvinov opened this issue Sep 20, 2022 · 3 comments · Fixed by #48797
Labels
Apply Apply, Aggregate, Transform, Map Regression Functionality that used to work in a prior pandas version
Milestone

Comments

@vnlitvinov
Copy link
Contributor

Pandas version checks

  • I have checked that this issue has not already been reported.

  • I have confirmed this bug exists on the latest version of pandas.

  • I have confirmed this bug exists on the main branch of pandas.

Reproducible Example

import pandas as pd
df = pd.DataFrame({'a': [0, 1]})
print(df.apply('any', axis=1))

Issue Description

When running the snippet above, pandas 1.5 produces the following stack trace:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\Vass\.conda\envs\pandas\lib\site-packages\pandas\core\frame.py", line 9558, in apply
    return op.apply().__finalize__(self, method="apply")
  File "C:\Users\Vass\.conda\envs\pandas\lib\site-packages\pandas\core\apply.py", line 720, in apply
    return self.apply_str()
  File "C:\Users\Vass\.conda\envs\pandas\lib\site-packages\pandas\core\apply.py", line 923, in apply_str
    return super().apply_str()
  File "C:\Users\Vass\.conda\envs\pandas\lib\site-packages\pandas\core\apply.py", line 556, in apply_str
    raise ValueError(f"Operation {f} does not support axis=1")
ValueError: Operation any does not support axis=1

I believe the root cause is this chunk of code:

pandas/pandas/core/apply.py

Lines 550 to 556 in c68a96f

func = getattr(obj, f, None)
if callable(func):
sig = inspect.getfullargspec(func)
if self.axis != 0 and (
"axis" not in sig.args or f in ("corrwith", "mad", "skew")
):
raise ValueError(f"Operation {f} does not support axis=1")

which should get the signature of pandas.DataFrame.any but, as .any() is decorated:

pandas/pandas/core/generic.py

Lines 11629 to 11647 in c68a96f

@deprecate_nonkeyword_arguments(
version=None,
allowed_args=["self"],
name="DataFrame.any and Series.any",
)
@doc(
_bool_doc,
desc=_any_desc,
name1=name1,
name2=name2,
axis_descr=axis_descr,
see_also=_any_see_also,
examples=_any_examples,
empty_value=False,
)
def any(
self, axis=0, bool_only=None, skipna: bool_t = True, level=None, **kwargs
):
return NDFrame.any(self, axis, bool_only, skipna, level, **kwargs)

it gets the signature of the decorator function instead.

Expected Behavior

The snippet above is printing the following when run on pandas 1.4.4:

0    False
1     True
dtype: bool

I believe it should print the same on 1.5.x, too.

Installed Versions

pd.show_versions()
INSTALLED VERSIONS
------------------
commit           : 87cfe4e38bafe7300a6003a1d18bd80f3f77c763
python           : 3.8.13.final.0
python-bits      : 64
OS               : Windows
OS-release       : 10
Version          : 10.0.22000
machine          : AMD64
processor        : AMD64 Family 25 Model 80 Stepping 0, AuthenticAMD
byteorder        : little
LC_ALL           : None
LANG             : None
LOCALE           : Russian_Russia.1251

pandas           : 1.5.0
numpy            : 1.23.3
pytz             : 2022.2.1
dateutil         : 2.8.2
setuptools       : 65.3.0
pip              : 22.2.2
Cython           : None
pytest           : None
hypothesis       : None
sphinx           : None
blosc            : None
feather          : None
xlsxwriter       : None
lxml.etree       : None
html5lib         : None
pymysql          : None
psycopg2         : None
jinja2           : None
IPython          : None
pandas_datareader: None
bs4              : None
bottleneck       : None
brotli           : None
fastparquet      : None
fsspec           : None
gcsfs            : None
matplotlib       : None
numba            : None
numexpr          : None
odfpy            : None
openpyxl         : None
pandas_gbq       : None
pyarrow          : None
pyreadstat       : None
pyxlsb           : None
s3fs             : None
scipy            : None
snappy           : None
sqlalchemy       : None
tables           : None
tabulate         : None
xarray           : None
xlrd             : None
xlwt             : None
zstandard        : None
tzdata           : None
@vnlitvinov vnlitvinov added Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Sep 20, 2022
@MarcoGorelli MarcoGorelli added the Regression Functionality that used to work in a prior pandas version label Sep 20, 2022
@MarcoGorelli
Copy link
Member

Thanks @vnlitvinov for the report, looks like this was caused by #44896

@phofl
Copy link
Member

phofl commented Sep 20, 2022

Yep, can confirm:

commit a59582da8c9fa410ac05e25ee117a7fef996247f
Author: Sachin Yadav <40866128+yadav-sachin@users.noreply.github.com>
Date:   Mon Apr 11 01:04:27 2022 +0530

    DEPR: non-keyword arguments in any (#44896)

@phofl phofl added this to the 1.5.1 milestone Sep 20, 2022
@MarcoGorelli MarcoGorelli removed Bug Needs Triage Issue that has not been reviewed by a pandas team member labels Sep 20, 2022
@phofl phofl added the Apply Apply, Aggregate, Transform, Map label Sep 20, 2022
hauntsaninja added a commit to hauntsaninja/pandas that referenced this issue Sep 26, 2022
Fixes pandas-dev#48656. Builds on top of pandas-dev#48693.

An alternative fix could be to use `inspect.signature`, since
`inspect.signature` defaults to following wrapped functions. But this
seems like a more minimal change.
@MarcoGorelli
Copy link
Member

Hey @vnlitvinov

Thanks for having reported this - there's now a release candidate for pandas 2.0.0, which you can install with

mamba install -c conda-forge/label/pandas_rc pandas==2.0.0rc0

or

python -m pip install --upgrade --pre pandas==2.0.0rc0

If you try it out and report bugs, then we can fix them before the final 2.0.0 release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Apply Apply, Aggregate, Transform, Map Regression Functionality that used to work in a prior pandas version
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants