Skip to content

Commit

Permalink
Test a fix for the libcugraph_etl build issues from libcudf
Browse files Browse the repository at this point in the history
  • Loading branch information
vyasr committed Feb 5, 2024
1 parent f77bee5 commit 8a0d386
Showing 1 changed file with 9 additions and 7 deletions.
16 changes: 9 additions & 7 deletions cpp/libcugraph_etl/src/renumbering.cu
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2022-2023, NVIDIA CORPORATION.
* Copyright (c) 2022-2024, NVIDIA CORPORATION.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
Expand Down Expand Up @@ -776,15 +776,15 @@ struct renumber_functor {
for (int i = 0; i < src_view.num_columns(); i++) {
auto str_col_view = cudf::strings_column_view(src_view.column(i));
src_vertex_chars_ptrs.push_back(
const_cast<char_type*>(str_col_view.chars().data<char_type>()));
const_cast<char_type*>(str_col_view.parent().data<char_type>()));
src_vertex_offset_ptrs.push_back(
const_cast<str_offset_type*>(str_col_view.offsets().data<str_offset_type>()));
}

for (int i = 0; i < dst_view.num_columns(); i++) {
auto str_col_view = cudf::strings_column_view(dst_view.column(i));
dst_vertex_chars_ptrs.push_back(
const_cast<char_type*>(str_col_view.chars().data<char_type>()));
const_cast<char_type*>(str_col_view.parent().data<char_type>()));
dst_vertex_offset_ptrs.push_back(
const_cast<str_offset_type*>(str_col_view.offsets().data<str_offset_type>()));
}
Expand Down Expand Up @@ -970,13 +970,14 @@ struct renumber_functor {
std::move(unrenumber_col1_chars),
rmm::device_buffer{},
0);
auto str_col_1_contents = str_col_1->release();

renumber_table_vectors.push_back(
cudf::make_strings_column(size_type(key_value_count),
std::move(offset_col_1),
std::move(str_col_1),
std::move(*str_col_1_contents.data),
0,
rmm::device_buffer(size_type(0), exec_strm)));
std::move(*str_col_1_contents.null_mask)));

auto offset_col_2 =
std::make_unique<cudf::column>(cudf::data_type(cudf::type_id::INT32),
Expand All @@ -991,13 +992,14 @@ struct renumber_functor {
std::move(unrenumber_col2_chars),
rmm::device_buffer{},
0);
auto str_col_2_contents = str_col_2->release();

renumber_table_vectors.push_back(
cudf::make_strings_column(size_type(key_value_count),
std::move(offset_col_2),
std::move(str_col_2),
std::move(*str_col_2_contents.data),
0,
rmm::device_buffer(size_type(0), exec_strm)));
std::move(*str_col_2_contents.null_mask)));

// make table from string columns - did at the end

Expand Down

0 comments on commit 8a0d386

Please sign in to comment.