Skip to content

Commit

Permalink
opt out of tensor-table support by default
Browse files Browse the repository at this point in the history
  • Loading branch information
rjzamora committed Apr 17, 2024
1 parent f4c9de3 commit 700b196
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
6 changes: 1 addition & 5 deletions nvtabular/ops/groupby.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
from merlin.core.dispatch import DataFrameType, annotate
from merlin.dtypes.shape import DefaultShapes
from merlin.schema import Schema
from nvtabular.ops.operator import ColumnSelector, DataFormats, Operator
from nvtabular.ops.operator import ColumnSelector, Operator


class Groupby(Operator):
Expand Down Expand Up @@ -111,10 +111,6 @@ def __init__(
self.name_sep = name_sep
super().__init__()

@property
def supported_formats(self):
return DataFormats.PANDAS_DATAFRAME | DataFormats.CUDF_DATAFRAME

@annotate("Groupby_op", color="darkgreen", domain="nvt_python")
def transform(self, col_selector: ColumnSelector, df: DataFrameType) -> DataFrameType:
# Sort if necessary
Expand Down
7 changes: 6 additions & 1 deletion nvtabular/ops/operator.py
Original file line number Diff line number Diff line change
Expand Up @@ -19,4 +19,9 @@
DataFormats,
)

Operator = BaseOperator

# Avoid TENSOR_TABLE by default (for now)
class Operator(BaseOperator):
@property
def supported_formats(self):
return DataFormats.PANDAS_DATAFRAME | DataFormats.CUDF_DATAFRAME

0 comments on commit 700b196

Please sign in to comment.