Skip to content

Commit

Permalink
Merge branch 'branch-25.02' into more_cccl_fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
miscco committed Jan 30, 2025
2 parents 8ddc1df + 0f61e37 commit d678e33
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 18 deletions.
2 changes: 1 addition & 1 deletion python/cudf_polars/cudf_polars/dsl/expressions/datetime.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,8 +57,8 @@ class Name(IntEnum):
OffsetBy = auto()
OrdinalDay = auto()
Quarter = auto()
ReplaceTimeZone = auto()
Replace = auto()
ReplaceTimeZone = auto()
Round = auto()
Second = auto()
Time = auto()
Expand Down
12 changes: 3 additions & 9 deletions python/cudf_polars/cudf_polars/dsl/ir.py
Original file line number Diff line number Diff line change
Expand Up @@ -696,14 +696,12 @@ class DataFrameScan(IR):
This typically arises from ``q.collect().lazy()``
"""

__slots__ = ("config_options", "df", "predicate", "projection")
_non_child = ("schema", "df", "projection", "predicate", "config_options")
__slots__ = ("config_options", "df", "projection")
_non_child = ("schema", "df", "projection", "config_options")
df: Any
"""Polars LazyFrame object."""
projection: tuple[str, ...] | None
"""List of columns to project out."""
predicate: expr.NamedExpr | None
"""Mask to apply."""
config_options: dict[str, Any]
"""GPU-specific configuration options"""

Expand All @@ -712,15 +710,13 @@ def __init__(
schema: Schema,
df: Any,
projection: Sequence[str] | None,
predicate: expr.NamedExpr | None,
config_options: dict[str, Any],
):
self.schema = schema
self.df = df
self.projection = tuple(projection) if projection is not None else None
self.predicate = predicate
self.config_options = config_options
self._non_child_args = (schema, df, self.projection, predicate)
self._non_child_args = (schema, df, self.projection)
self.children = ()

def get_hashable(self) -> Hashable:
Expand All @@ -736,7 +732,6 @@ def get_hashable(self) -> Hashable:
schema_hash,
id(self.df),
self.projection,
self.predicate,
json.dumps(self.config_options),
)

Expand All @@ -746,7 +741,6 @@ def do_evaluate(
schema: Schema,
df: Any,
projection: tuple[str, ...] | None,
predicate: expr.NamedExpr | None,
) -> DataFrame:
"""Evaluate and return a dataframe."""
pdf = pl.DataFrame._from_pydf(df)
Expand Down
3 changes: 0 additions & 3 deletions python/cudf_polars/cudf_polars/dsl/translate.py
Original file line number Diff line number Diff line change
Expand Up @@ -260,9 +260,6 @@ def _(
schema,
node.df,
node.projection,
translate_named_expr(translator, n=node.selection)
if node.selection is not None
else None,
translator.config.config.copy(),
)

Expand Down
3 changes: 1 addition & 2 deletions python/cudf_polars/cudf_polars/experimental/io.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# SPDX-FileCopyrightText: Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES.
# SPDX-FileCopyrightText: Copyright (c) 2024-2025, NVIDIA CORPORATION & AFFILIATES.
# SPDX-License-Identifier: Apache-2.0
"""Multi-partition IO Logic."""

Expand Down Expand Up @@ -42,7 +42,6 @@ def _(
ir.schema,
ir.df.slice(offset, length),
ir.projection,
ir.predicate,
ir.config_options,
)
for offset in range(0, nrows, length)
Expand Down
4 changes: 1 addition & 3 deletions python/cudf_polars/tests/dsl/test_traversal.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# SPDX-FileCopyrightText: Copyright (c) 2024 NVIDIA CORPORATION & AFFILIATES.
# SPDX-FileCopyrightText: Copyright (c) 2024-2025, NVIDIA CORPORATION & AFFILIATES.
# SPDX-License-Identifier: Apache-2.0

from __future__ import annotations
Expand Down Expand Up @@ -119,7 +119,6 @@ def replace_df(node, rec):
node.schema,
new_df._df,
node.projection,
node.predicate,
node.config_options,
)
return reuse_if_unchanged(node, rec)
Expand Down Expand Up @@ -151,7 +150,6 @@ def replace_scan(node, rec):
node.schema,
right._df,
node.with_columns,
node.predicate,
node.config_options,
)
return reuse_if_unchanged(node, rec)
Expand Down

0 comments on commit d678e33

Please sign in to comment.