Skip to content

Commit

Permalink
cleanup
Browse files Browse the repository at this point in the history
Signed-off-by: Anatoly Myachev <[email protected]>
  • Loading branch information
anmyachev committed May 14, 2024
1 parent 18eec16 commit b3471ff
Show file tree
Hide file tree
Showing 6 changed files with 13 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ class PandasOnDaskDataframe(PandasDataframe):
each of the block partitions. Is computed if not provided.
dtypes : pandas.Series, optional
The data types for the dataframe columns.
pandas_backend : {"pyarrow", None}, optional
Backend used by pandas. None - means default NumPy backend.
"""

_partition_mgr_cls = PandasOnDaskDataframePartitionManager
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@ class PandasOnPythonDataframe(PandasDataframe):
each of the block partitions. Is computed if not provided.
dtypes : pandas.Series, optional
The data types for the dataframe columns.
pandas_backend : {"pyarrow", None}, optional
Backend used by pandas. None - means default NumPy backend.
"""

_partition_mgr_cls = PandasOnPythonDataframePartitionManager
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@ class PandasOnRayDataframe(PandasDataframe):
each of the block partitions. Is computed if not provided.
dtypes : pandas.Series, optional
The data types for the dataframe columns.
pandas_backend : {"pyarrow", None}, optional
Backend used by pandas. None - means default NumPy backend.
"""

_partition_mgr_cls = PandasOnRayDataframePartitionManager
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ class PandasOnUnidistDataframe(PandasDataframe):
each of the block partitions. Is computed if not provided.
dtypes : pandas.Series, optional
The data types for the dataframe columns.
pandas_backend : {"pyarrow", None}, optional
Backend used by pandas. None - means default NumPy backend.
"""

_partition_mgr_cls = PandasOnUnidistDataframePartitionManager
Expand Down
12 changes: 5 additions & 7 deletions modin/core/storage_formats/base/query_compiler.py
Original file line number Diff line number Diff line change
Expand Up @@ -1118,7 +1118,7 @@ def merge_asof(
tolerance=None,
allow_exact_matches: bool = True,
direction: str = "backward",
): # noqa: GL08
):
# Pandas fallbacks for tricky cases:
if (
# No idea how this works or why it does what it does; and in fact
Expand Down Expand Up @@ -3622,9 +3622,7 @@ def groupby_fillna(
drop=drop,
)

def groupby_diff(
self, by, axis, groupby_kwargs, agg_args, agg_kwargs, drop=False
): # noqa: GL08
def groupby_diff(self, by, axis, groupby_kwargs, agg_args, agg_kwargs, drop=False):
return self.groupby_agg(
by=by,
agg_func="diff",
Expand All @@ -3637,7 +3635,7 @@ def groupby_diff(

def groupby_pct_change(
self, by, axis, groupby_kwargs, agg_args, agg_kwargs, drop=False
): # noqa: GL08
):
return self.groupby_agg(
by=by,
agg_func="pct_change",
Expand Down Expand Up @@ -3945,7 +3943,7 @@ def groupby_ohlc(
agg_args,
agg_kwargs,
is_df,
): # noqa: GL08
):
if not is_df:
return self.groupby_agg(
by=by,
Expand Down Expand Up @@ -4609,7 +4607,7 @@ def shift(
freq,
axis,
fill_value,
): # noqa: GL08
):
return DataFrameDefault.register(pandas.DataFrame.shift)(
self, periods, freq, axis, fill_value
)
Expand Down
3 changes: 0 additions & 3 deletions modin/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -310,9 +310,6 @@ def _replace_doc(
target_doc = target_obj.__doc__ or ""
overwrite = overwrite or not target_doc
doc = source_doc if overwrite else target_doc
if doc == "":
# Empty docstrings do not need to be inherited
return

if parent_cls and not attr_name:
if isinstance(target_obj, property):
Expand Down

0 comments on commit b3471ff

Please sign in to comment.