-
Notifications
You must be signed in to change notification settings - Fork 651
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
REFACTOR-#7294: Reduce access of methods _modin_frame methods from query compiler objects #7297
Conversation
4ba54e8
to
b8d07a7
Compare
Returns | ||
------- | ||
bool | ||
True if if the undelying dataframe and False otherwise. |
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.
True if if the undelying dataframe and False otherwise. |
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.
Removed
@@ -4521,6 +4521,60 @@ def has_multiindex(self, axis=0): | |||
assert axis == 1 | |||
return isinstance(self.columns, pandas.MultiIndex) | |||
|
|||
@property | |||
def has_materialized_dtypes(self) -> bool: |
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.
By analogy
def has_materialized_dtypes(self) -> bool: | |
def frame_has_materialized_dtypes(self) -> bool: |
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.
Done
…thods from _query_compiler
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.
LGTM!
@@ -4521,6 +4521,59 @@ def has_multiindex(self, axis=0): | |||
assert axis == 1 | |||
return isinstance(self.columns, pandas.MultiIndex) | |||
|
|||
@property | |||
def frame_has_materialized_dtypes(self) -> bool: |
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.
@anmyachev, @arunjose696, why didn't we add similar methods for operating on columns?
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.
Apparently the relevant code was simply not touched when working on the small query compiler. There is no reason not to do this for columns.
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 would propose to do this for consistency. @arunjose696, would you take on this?
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