-
Notifications
You must be signed in to change notification settings - Fork 304
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
update conv layers in cugraph-dgl for pylibcugraphops 23.04 #3360
update conv layers in cugraph-dgl for pylibcugraphops 23.04 #3360
Conversation
@@ -117,7 +117,7 @@ if [[ "${RAPIDS_CUDA_VERSION}" == "11.8.0" ]]; then | |||
--channel "${PYTHON_CHANNEL}" \ | |||
--channel pytorch \ | |||
--channel pytorch-nightly \ | |||
--channel dglteam/label/cu117 \ | |||
--channel dglteam/label/cu118 \ |
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.
Ahh, just saw that that they have cu118 builds here: https://anaconda.org/dglteam/dgl
Surprisingly it is missing on getting started page: https://www.dgl.ai/pages/start.html
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 from my side. as discussed live, with 23.06, hopefully, we can get rid of the pad_offsets
thing altogether, using new structures
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.
Approving ops-codeowner
file changes
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
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
/merge |
Update cugraph-ops models to use pylibcugraphops 23.04. This PR also supersedes #3264
CC: @MatthiasKohl