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.

Sign up for GitHub

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

Jump to bottom

Array.reindex silently ignores new_axis argument when first argument is an Axis #1121

Open

gdementen opened this issue Oct 24, 2024 · 0 comments

Labels

bug difficulty: low priority: very high size: small

Milestone

Contributor

gdementen commented Oct 24, 2024

While looking at the code of reindex for #1120 , I noticed the following bug:

>>> arr = ndtest(3)
>>> arr.reindex("a", "a1,a2,a3")
a   a1   a2   a3
   1.0  2.0  nan
>>> arr.reindex(arr.a, "a1,a2,a3")
a   a0   a1   a2
   0.0  1.0  2.0

Very high priority because this is a silent failure. I think fixing this should only be a matter of adding "and new_axis is None".

gdementen added bug difficulty: low size: small priority: very high labels

gdementen added this to the 0.35 milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment