-
-
Notifications
You must be signed in to change notification settings - Fork 17.9k
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: unexpected merge_ordered results caused by wrongly groupby #38170
BUG: unexpected merge_ordered results caused by wrongly groupby #38170
Conversation
@@ -274,10 +271,20 @@ def _merger(x, y): | |||
if left_by is not None and right_by is not None: | |||
raise ValueError("Can only group either left or right frames") | |||
elif left_by is not None: | |||
if isinstance(left_by, str): |
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.
its possible that we want to integrate these checks much more into: https://github.com/pandas-dev/pandas/blob/master/pandas/core/reshape/merge.py#L1247
but this may require some refactoring. (as a followup if you can)
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.
I will take a try.
thanks @GYHHAHA very nice |
black pandas
git diff upstream/master -u -- "*.py" | flake8 --diff