Skip to content

Commit 8c92384

Browse files
committed
Misc feedback
1 parent 521ca70 commit 8c92384

File tree

3 files changed

+6
-4
lines changed

3 files changed

+6
-4
lines changed

modin/core/storage_formats/base/query_compiler.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ def axis_setter(self, labels):
110110

111111
class QCCoercionCost(IntEnum): # noqa: PR01
112112
"""
113-
Coercion costs between Query Compilers.
113+
Coercion costs between different Query Compiler backends.
114114
115115
Coercion costs between query compilers can be expressed
116116
as integers in the range 0 to 1000, where 1000 is
@@ -125,7 +125,7 @@ class QCCoercionCost(IntEnum): # noqa: PR01
125125
126126
COST_IMPOSSIBLE means the coercion is effectively impossible, which can
127127
occur if the target system is unable to store the data as a result
128-
of the coercion.
128+
of the coercion. Currently this does not prevent coercion.
129129
"""
130130

131131
COST_ZERO = 0

modin/core/storage_formats/pandas/native_query_compiler.py

+3-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@
2828
)
2929
from modin.core.storage_formats.base.query_compiler import BaseQueryCompiler
3030
from modin.core.storage_formats.pandas.query_compiler_caster import QueryCompilerCaster
31-
from modin.utils import _inherit_docstrings
31+
from modin.utils import (
32+
_inherit_docstrings,
33+
)
3234

3335
_NO_REPARTITION_ON_NATIVE_EXECUTION_EXCEPTION_MESSAGE = (
3436
"Modin dataframes and series using native execution do not have partitions."

modin/core/storage_formats/pandas/query_compiler_caster.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ def __init__(self):
5151
self._qc_cls_set = set()
5252
self._result_type = None
5353

54-
def add_query_compiler(self, query_compiler):
54+
def add_query_compiler(self, query_compiler: BaseQueryCompiler):
5555
"""
5656
Add a query compiler to be considered for casting.
5757

0 commit comments

Comments
 (0)