-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
BUG: RangeIndex accepting length-1 arrays as argument #13765
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
BUG: RangeIndex accepting length-1 arrays as argument #13765
Conversation
48506ca
to
440d262
Compare
Current coverage is 85.27% (diff: 100%)@@ master #13765 diff @@
==========================================
Files 139 139
Lines 50211 50214 +3
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
+ Hits 42819 42820 +1
- Misses 7392 7394 +2
Partials 0 0
|
440d262
to
6180487
Compare
@jreback OK? |
can you add the orginal issue example somewhere (and assert it works) |
That's what I added in |
is it a simplified example? the original one is on a panel |
Well, looking back at the original issue #13542, this is indeed not the original example I included in the tests. But I found the reported example rather convoluted/difficult to comprehend what was going on. And the example I added, is a simplified (whithout using panels) example that experience exactly the same issue (it is what happened inside the groupby). But I can add the original panel example as well if you like |
it's fine |
git diff upstream/master | flake8 --diff