Skip to content

Commit

Permalink
a few doc changes based on PR comments
Browse files Browse the repository at this point in the history
  • Loading branch information
ChuckHastings committed May 13, 2022
1 parent d8f69c8 commit bad27dc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cpp/include/cugraph/algorithms.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -1547,8 +1547,8 @@ uniform_nbr_sample(raft::handle_t const& handle,
* @param handle RAFT handle object to encapsulate resources (e.g. CUDA stream, communicator, and
* handles to various CUDA libraries) to run graph algorithms.
* @param graph_view Graph View object to generate NBR Sampling on.
* @param d_starting_vertices Device span of starting vertex IDs for the NBR Sampling.
* @param h_fan_out Host span defining branching out (fan-out) degree per source vertex for each
* @param starting_vertices Device span of starting vertex IDs for the NBR Sampling.
* @param fan_out Host span defining branching out (fan-out) degree per source vertex for each
* level
* @param with_replacement boolean flag specifying if random sampling is done with replacement
* (true); or, without replacement (false); default = true;
Expand Down
3 changes: 3 additions & 0 deletions cpp/include/cugraph/detail/decompress_edge_partition.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -362,6 +362,9 @@ void partially_decompress_edge_partition_to_fill_edgelist(
edge_t local_degree{};
thrust::tie(indices, weights, local_degree) =
edge_partition.local_edges(major_partition_offset);

// FIXME: This can lead to thread divergence if local_degree varies significantly
// within threads in this warp
thrust::fill(
thrust::seq, majors + major_offset, majors + major_offset + local_degree, major);
thrust::copy(thrust::seq, indices, indices + local_degree, minors + major_offset);
Expand Down

0 comments on commit bad27dc

Please sign in to comment.