Skip to content

Commit

Permalink
Fix disjoint mode to only operate in temporal sampling cases (#8482)
Browse files Browse the repository at this point in the history
  • Loading branch information
rusty1s authored Nov 29, 2023
1 parent 12a2fb2 commit 1d11179
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions torch_geometric/distributed/dist_neighbor_sampler.py
Original file line number Diff line number Diff line change
Expand Up @@ -619,12 +619,12 @@ def _sample_one_hop(
True, # csc
self.replace,
self.subgraph_type != SubgraphType.induced,
self.disjoint or self.time_attr is not None,
self.disjoint and self.time_attr is not None,
self.temporal_strategy,
)
node, edge, cumsum_neighbors_per_node = out

if self.disjoint or self.time_attr is not None:
if self.disjoint and self.time_attr is not None:
# We create a batch during the step of merging sampler outputs.
_, node = node.t().contiguous()

Expand Down

0 comments on commit 1d11179

Please sign in to comment.