-
Notifications
You must be signed in to change notification settings - Fork 96
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
Install Dask + Distributed from main
#546
Conversation
These recently dropped the `master` branch and switched to `main`. So update the install steps to use `main` instead.
@gpucibot merge |
Seems like we are getting some explicit comms test failures. @rjzamora would you be able to take a look? 🙂 |
The issues seem to be GPU-related in CI:
Rerunning to see if this unblocks. |
rerun tests |
Seeing this in the log
Also seeing this
|
Wondering if that last part is related to PR ( dask/distributed#4531 ) cc @madsbk (in case you have any thoughts here 🙂) |
Ah sorry @jakirkham , I missed those errors, but I'm seeing them now too on the latest CI run as well. Seems like a potential issue coming from dask/distributed#4531 indeed, so it would be good to have @madsbk looking into it. To unblock CI, what do you think about xfailing those tests @jakirkham ? |
Sure marked as |
0158a4d
to
f520186
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.
LGTM, thanks @jakirkham !
@jakirkham seems like
|
f520186
to
2edab05
Compare
Good catch! Thanks Peter 😄 Sorry had missed that earlier. Should be addressed now 🙂 |
These recently dropped the
master
branch and switched tomain
. So update the install steps to usemain
instead.