-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Replaced xrange with range #30417
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
Replaced xrange with range #30417
Conversation
@@ -585,10 +585,6 @@ def test_strange_column_corruption_issue(self): | |||
df = DataFrame(index=[0, 1]) | |||
df[0] = np.nan | |||
wasCol = {} | |||
# uncommenting these makes the results match | |||
# for col in xrange(100, 200): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This code block is sitting commented for over 4 years. thtat's the only reason I deleted it all. and not just changed the xrange
to range
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Any idea why xrange even worked in the pyx file? I would presume ot should fail when we`re on a python3 code base.
LGTM |
Nice - thanks @MomIsBestFriend |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff