-
Notifications
You must be signed in to change notification settings - Fork 644
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FIX-#6201: align groupby objects signatures with pandas #6202
Conversation
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, left minor comments
modin/pandas/groupby.py
Outdated
if engine is not None and engine_kwargs is not None: | ||
return self._default_to_pandas( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should eventually do this for every method that can take the engine
parameters, probably as a part of a separate PR sometime in the future
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will do this right now.
Co-authored-by: Dmitry Chigarev <dmitry.chigarev@intel.com>
Signed-off-by: Anatoly Myachev <anatoly.myachev@intel.com>
@@ -278,12 +278,22 @@ def value_counts( | |||
) | |||
) | |||
|
|||
def mean(self, numeric_only=None): | |||
def mean(self, numeric_only=no_default, engine="cython", engine_kwargs=None): | |||
if engine not in ("cython", None) and engine_kwargs is not None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cython
is the default value that pandas converts None
value to. For example, https://pandas.pydata.org/pandas-docs/version/1.5.3/reference/api/pandas.core.groupby.GroupBy.std.html?highlight=std#pandas.core.groupby.GroupBy.std
What do these changes do?
flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
git commit -s
docs/development/architecture.rst
is up-to-date