dmlc / dgl

Python package built to ease deep learning on graph, on top of existing DL frameworks.
http://dgl.ai
Apache License 2.0
13.23k stars 2.99k forks source link

[dev] update minimum torch version to 2.1.0 #7401

Closed Rhett-Ying closed 2 months ago

Rhett-Ying commented 2 months ago

Description

Checklist

Please feel free to remove inapplicable items for your PR.

Changes

dgl-bot commented 2 months ago

To trigger regression tests:

dgl-bot commented 2 months ago

Commit ID: 91650484912dada2181d66068bc9a776283a62b7

Build ID: 1

Status: ❌ CI test failed in Stage [PyTorch Cugraph GPU Unit test].

Report path: link

Full logs path: link

Rhett-Ying commented 2 months ago

@TristonC

We're updating the minimum pytorch version from 2.0.0 to 2.1.0 and CI is blocked by cugraph docker image: rapidsai/cugraph_stable_torch-cuda:11.8-base-ubuntu20.04-py3.10-pytorch2.0.0-rapids23.04.

On the web page: https://hub.docker.com/r/rapidsai/cugraph_stable_torch-cuda, I cannot find any newer ones.

So do you have ideas about how to handle this?

frozenbugs commented 2 months ago

@dgl-bot

dgl-bot commented 2 months ago

Commit ID: 91650484912dada2181d66068bc9a776283a62b7

Build ID: 2

Status: ❌ CI test failed in Stage [PyTorch Cugraph GPU Unit test].

Report path: link

Full logs path: link

Rhett-Ying commented 2 months ago

@TristonC

We're updating the minimum pytorch version from 2.0.0 to 2.1.0 and CI is blocked by cugraph docker image: rapidsai/cugraph_stable_torch-cuda:11.8-base-ubuntu20.04-py3.10-pytorch2.0.0-rapids23.04.

On the web page: https://hub.docker.com/r/rapidsai/cugraph_stable_torch-cuda, I cannot find any newer ones.

So do you have ideas about how to handle this?

@chang-l Hi, do you have any suggestion on this? Should we bypass the test on cugraph or upgrade docker image?

chang-l commented 2 months ago

@Rhett-Ying, can you please try switch to NVIDIA's public NGC DGL container for your CI/CD?(https://catalog.ngc.nvidia.com/orgs/nvidia/containers/dgl) docker pull nvcr.io/nvidia/dgl:24.04-py3

It includes latest PyTorch and latest RAPIDS (e.g., cugraph/wholegraph) for graph analysis.

TristonC commented 2 months ago

@Rhett-Ying We can pause (bypass) the cugraph test for now. cugraph is in the process of refactoring. We can add the test back once its ready. Or as @chang-l suggested to use NVIDIA DGL container.

Rhett-Ying commented 2 months ago

@Rhett-Ying We can pause (bypass) the cugraph test for now. cugraph is in the process of refactoring. We can add the test back once its ready. Or as @chang-l suggested to use NVIDIA DGL container.

Sure. Let's pause the cugraph test for now.

And is there any ticket link to track the status of cugraph refactoring? we could link it here.

dgl-bot commented 2 months ago

Commit ID: cb19e6bc4730f67057122ee707d31fdf9513b338

Build ID: 3

Status: ✅ CI test succeeded.

Report path: link

Full logs path: link