Hongzhi (Steve), Chen

``` ************* Module dgl.dataloading.capped_neighbor_sampler python/dgl/dataloading/capped_neighbor_sampler.py:60:4: W0221: Parameters differ from overridden 'sample' method (arguments-differ) python/dgl/dataloading/capped_neighbor_sampler.py:151:27: C0103: Variable name "node_IDs" doesn't conform to snake_case naming style (invalid-name) python/dgl/dataloading/capped_neighbor_sampler.py:102:27: W0612: Unused variable 'rel_type'...

@ayushnoori if you don't plan to write a unit test, we can also merge this, can you add a comment in capped_neighbor_sampler.py, says "This code was contributed by a community...

Node classification and multi-GPU done, missing link prediction.

> However, how do you think we can check the graph in a general manner? The user might pass sample_neighbors, sample_layer_neighbors or any other custom datapipes. The features could be...

Hi @ayushnoori as DGL Graphbolt release, I am closing this as it, please try our new dataloading package DGL Graphbolt, let us know if GB does not solve your problem.

https://github.com/dmlc/dgl/blob/41a38486a5ed9298093d9f0bc415751269c7d577/python/dgl/graphbolt/dataloader.py#L58

Let's aim GS 0.3 release around July 2024, before the release there will be comprehensive test.

> @frozenbugs do we or GS run comprehensive test over this PR? As I just disabled the whole distributed tests, please run them manually. GS will do it.

Archived: It is probably not an option to pin torchdata < 0.8 since it has dependency on torch core, which will lead to dependency on 2 different torch version https://github.com/dmlc/dgl/pull/7604.

> #7609 has removed torchdata imports from GraphBolt. We still need to remove torchdata as a dependency. Pending https://github.com/dmlc/dgl/pull/7667