Skip to content

Commit

Permalink
Remove cudf._lib.json in favor of inlining pylibcudf (#17443)
Browse files Browse the repository at this point in the history
Contributes to #17317

Authors:
  - Matthew Roeschke (https://github.com/mroeschke)

Approvers:
  - Matthew Murray (https://github.com/Matt711)

URL: #17443
  • Loading branch information
mroeschke authored Nov 28, 2024
1 parent 2be82fe commit 9b88794
Show file tree
Hide file tree
Showing 7 changed files with 238 additions and 246 deletions.
1 change: 0 additions & 1 deletion python/cudf/cudf/_lib/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ set(cython_sources
filling.pyx
groupby.pyx
interop.pyx
json.pyx
merge.pyx
orc.pyx
parquet.pyx
Expand Down
1 change: 0 additions & 1 deletion python/cudf/cudf/_lib/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
filling,
groupby,
interop,
json,
merge,
nvtext,
orc,
Expand Down
211 changes: 0 additions & 211 deletions python/cudf/cudf/_lib/json.pyx

This file was deleted.

2 changes: 1 addition & 1 deletion python/cudf/cudf/_lib/utils.pxd
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,4 @@ cdef table_view table_view_from_table(tbl, ignore_index=*) except*
cdef columns_from_unique_ptr(unique_ptr[table] c_tbl)
cdef columns_from_table_view(table_view tv, object owners)
cpdef columns_from_pylibcudf_table(tbl)
cdef _data_from_columns(columns, column_names, index_names=*)
cpdef _data_from_columns(columns, column_names, index_names=*)
2 changes: 1 addition & 1 deletion python/cudf/cudf/_lib/utils.pyx
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ cpdef columns_from_pylibcudf_table(tbl):
return [Column.from_pylibcudf(plc) for plc in tbl.columns()]


cdef _data_from_columns(columns, column_names, index_names=None):
cpdef _data_from_columns(columns, column_names, index_names=None):
"""Convert a list of columns into a dict with an index.
This method is intended to provide the bridge between the columns returned
Expand Down
Loading

0 comments on commit 9b88794

Please sign in to comment.