You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the solution you'd like and any additional context
We should allow sampling per edge type in uniform neighbor sample. After PR 2660 we have graph with type values.
I believe we should include an argument like do_fanout_per_type and sample fan_out number of edges for each seed node for every edge type.
Context:
DGL Behaviour:
DGL when sampling neighbors on Heterogeneous Graphs uses type information and by default samples fanout value per type . See below example.
fanout (int or dict[etype, int]) –
The number of edges to be sampled for each node on each edge type.
This argument can take a single int or a dictionary of edge types and ints.
If a single int is given, DGL will sample this number of edges "for each node for every edge type".
If -1 is given for a single edge type, all the neighboring edges with that edge type will be selected.
After PR 2660 we have graph with type values. #2660, we incude an argument like do_fanout_per_type and sample fanoout number of edges for each seed node for every edge type.
This issue has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this issue if no further response or action is needed. Otherwise, please respond with a comment indicating any updates or changes to the original issue and/or confirm this issue still needs to be addressed. This issue will be labeled inactive-90d if there is no activity in the next 60 days.
Describe the solution you'd like and any additional context
We should allow sampling per edge type in uniform neighbor sample. After PR 2660 we have graph with type values.
I believe we should include an argument like
do_fanout_per_type
and samplefan_out
number of edges for each seed node for every edge type.Context:
DGL Behaviour:
DGL when sampling neighbors on Heterogeneous Graphs uses type information and by default samples fanout value per type . See below example.
From DGL API docs (link)
cuGraph current behavior:
cuGraph requested feature
After PR 2660 we have graph with type values. #2660, we incude an argument like
do_fanout_per_type
and samplefanoout
number of edges for each seed node for every edge type.Alternate work-around
We store
n_etypes
views of the data and do the sampling call for each edge type individually which will be slow because we will maken_etypes
calls.CC: @ChuckHastings , @seunghwak , @alexbarghi-nv
CC: @rlratzel, @BradReesWork
The text was updated successfully, but these errors were encountered: