Skip to content

Commit

Permalink
Temporarily disable input argument checks for a currently disabled fe…
Browse files Browse the repository at this point in the history
…ature (#1840)

Temporarily disable input argument checks for meta data that are used for the currently disabled feature.

Authors:
  - Seunghwa Kang (https://github.com/seunghwak)

Approvers:
  - Rick Ratzel (https://github.com/rlratzel)
  - Chuck Hastings (https://github.com/ChuckHastings)

URL: #1840
  • Loading branch information
seunghwak authored Sep 23, 2021
1 parent 677b9f3 commit a728c49
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions cpp/src/structure/graph_impl.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -314,6 +314,13 @@ graph_t<vertex_t, edge_t, weight_t, store_transposed, multi_gpu, std::enable_if_
majors.begin(), thrust::unique(handle.get_thrust_policy(), majors.begin(), majors.end())));
auto num_local_unique_edge_minors = static_cast<vertex_t>(thrust::distance(
minors.begin(), thrust::unique(handle.get_thrust_policy(), minors.begin(), minors.end())));
// FIXME: temporarily disable this check as these are currently not used
// (row_col_properties_kv_pair_fill_ratio_threshold is set to 0.0, so (key, value) pairs for
// row/column properties will be never enabled) and we're not currently exposing this to the
// python layer. Should be re-enabled later once we enable the (key, value) pair feature and
// hopefully simplify the python graph creation pipeline as well (so no need to pass this
// information to the python layer).
#if 0
if constexpr (store_transposed) {
CUGRAPH_EXPECTS(num_local_unique_edge_majors == meta.num_local_unique_edge_cols,
"Invalid input argument: num_local_unique_edge_cols is erroneous.");
Expand All @@ -325,6 +332,7 @@ graph_t<vertex_t, edge_t, weight_t, store_transposed, multi_gpu, std::enable_if_
CUGRAPH_EXPECTS(num_local_unique_edge_minors == meta.num_local_unique_edge_cols,
"Invalid input argument: num_local_unique_edge_cols is erroneous.");
}
#endif
}

// aggregate segment_offsets
Expand Down

0 comments on commit a728c49

Please sign in to comment.