-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Fix #6775 #6809
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
Fix #6775 #6809
Conversation
cb3f2ca
to
b16e6b7
Compare
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 looks great! Thanks for opening the PR!
@@ -4141,6 +4141,8 @@ def join( | |||
npartitions=None, | |||
shuffle=None, | |||
): | |||
if is_series_like(other) and hasattr(other, "name"): | |||
other = other.to_frame() |
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.
It'd be nice to check for indexes here as well and convert those to dataframes.
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.
On Pandas
it will cause an error. (join indexes only using concat)
>>> df = pd.DataFrame({"x": [1, 2, 3, 4, 5, 6, 7, 8]})
>>> df.join(df.index)
---------------------------------------------------------------------------
AttributeError Traceback (most recent call last)
<ipython-input-31-5499579ebbb6> in <module>
----> 1 df.join(df.index)
~\anaconda3\lib\site-packages\pandas\core\frame.py in join(self, other, on, how, lsuffix, rsuffix, sort)
7206 5 K5 A5 NaN
7207 """
-> 7208 return self._join_compat(
7209 other, on=on, how=how, lsuffix=lsuffix, rsuffix=rsuffix, sort=sort
7210 )
~\anaconda3\lib\site-packages\pandas\core\frame.py in _join_compat(self, other, on, how, lsuffix, rsuffix, sort)
7240 frames = [self] + list(other)
7241
-> 7242 can_concat = all(df.index.is_unique for df in frames)
7243
7244 # join indexes only using concat
~\anaconda3\lib\site-packages\pandas\core\frame.py in <genexpr>(.0)
7240 frames = [self] + list(other)
7241
-> 7242 can_concat = all(df.index.is_unique for df in frames)
7243
7244 # join indexes only using concat
AttributeError: 'int' object has no attribute 'index'
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.
Ah ok! I guess this is good to go then :)
@dask/maintenance I don't have sufficient permissions to restart the travis job that timed out. |
I don't seem to have the rerun burron anymore either :| |
Sounds great, I will rebase on master and re-push. |
b16e6b7
to
a657a28
Compare
I'll merge when green :) |
Thanks for the PR @DavidKatz-il! |
Fix #6775
black dask
/flake8 dask