mirror of https://github.com/apache/superset.git
chore(pylint): Reenable too-many-locals check (#16268)
Co-authored-by: John Bodley <john.bodley@airbnb.com>
This commit is contained in:
parent
7b724439b9
commit
b5c7ed9f18
|
@ -87,7 +87,6 @@ disable=
|
||||||
import-outside-toplevel,
|
import-outside-toplevel,
|
||||||
raise-missing-from,
|
raise-missing-from,
|
||||||
too-few-public-methods,
|
too-few-public-methods,
|
||||||
too-many-locals,
|
|
||||||
duplicate-code,
|
duplicate-code,
|
||||||
|
|
||||||
[REPORTS]
|
[REPORTS]
|
||||||
|
|
|
@ -120,7 +120,7 @@ class QueryContext:
|
||||||
df.reset_index(level=0, inplace=True)
|
df.reset_index(level=0, inplace=True)
|
||||||
return df
|
return df
|
||||||
|
|
||||||
def processing_time_offsets(
|
def processing_time_offsets( # pylint: disable=too-many-locals
|
||||||
self, df: pd.DataFrame, query_object: QueryObject,
|
self, df: pd.DataFrame, query_object: QueryObject,
|
||||||
) -> CachedTimeOffset:
|
) -> CachedTimeOffset:
|
||||||
# ensure query_object is immutable
|
# ensure query_object is immutable
|
||||||
|
|
|
@ -671,7 +671,7 @@ class DashboardRestApi(BaseSupersetModelRestApi):
|
||||||
@event_logger.log_this_with_context(
|
@event_logger.log_this_with_context(
|
||||||
action=lambda self, *args, **kwargs: f"{self.__class__.__name__}.export",
|
action=lambda self, *args, **kwargs: f"{self.__class__.__name__}.export",
|
||||||
log_to_statsd=False,
|
log_to_statsd=False,
|
||||||
)
|
) # pylint: disable=too-many-locals
|
||||||
def export(self, **kwargs: Any) -> Response:
|
def export(self, **kwargs: Any) -> Response:
|
||||||
"""Export dashboards
|
"""Export dashboards
|
||||||
---
|
---
|
||||||
|
|
|
@ -56,7 +56,7 @@ def build_uuid_to_id_map(position: Dict[str, Any]) -> Dict[str, int]:
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
def update_id_refs(
|
def update_id_refs( # pylint: disable=too-many-locals
|
||||||
config: Dict[str, Any],
|
config: Dict[str, Any],
|
||||||
chart_ids: Dict[str, int],
|
chart_ids: Dict[str, int],
|
||||||
dataset_info: Dict[str, Dict[str, Any]],
|
dataset_info: Dict[str, Dict[str, Any]],
|
||||||
|
|
|
@ -415,7 +415,7 @@ class DatasetRestApi(BaseSupersetModelRestApi):
|
||||||
@event_logger.log_this_with_context(
|
@event_logger.log_this_with_context(
|
||||||
action=lambda self, *args, **kwargs: f"{self.__class__.__name__}.export",
|
action=lambda self, *args, **kwargs: f"{self.__class__.__name__}.export",
|
||||||
log_to_statsd=False,
|
log_to_statsd=False,
|
||||||
)
|
) # pylint: disable=too-many-locals
|
||||||
def export(self, **kwargs: Any) -> Response:
|
def export(self, **kwargs: Any) -> Response:
|
||||||
"""Export datasets
|
"""Export datasets
|
||||||
---
|
---
|
||||||
|
|
|
@ -32,7 +32,7 @@ from .helpers import (
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
def load_multiformat_time_series(
|
def load_multiformat_time_series( # pylint: disable=too-many-locals
|
||||||
only_metadata: bool = False, force: bool = False
|
only_metadata: bool = False, force: bool = False
|
||||||
) -> None:
|
) -> None:
|
||||||
"""Loading time series data from a zip file in the repo"""
|
"""Loading time series data from a zip file in the repo"""
|
||||||
|
|
|
@ -211,7 +211,7 @@ def _append_columns(
|
||||||
|
|
||||||
|
|
||||||
@validate_column_args("index", "columns")
|
@validate_column_args("index", "columns")
|
||||||
def pivot( # pylint: disable=too-many-arguments
|
def pivot( # pylint: disable=too-many-arguments,too-many-locals
|
||||||
df: DataFrame,
|
df: DataFrame,
|
||||||
index: List[str],
|
index: List[str],
|
||||||
aggregates: Dict[str, Dict[str, Any]],
|
aggregates: Dict[str, Dict[str, Any]],
|
||||||
|
|
|
@ -21,6 +21,7 @@ from unittest import mock
|
||||||
from werkzeug.wrappers import Request, Response
|
from werkzeug.wrappers import Request, Response
|
||||||
|
|
||||||
try:
|
try:
|
||||||
|
# pylint: disable=import-error
|
||||||
from pyinstrument import Profiler
|
from pyinstrument import Profiler
|
||||||
except ModuleNotFoundError:
|
except ModuleNotFoundError:
|
||||||
Profiler = None
|
Profiler = None
|
||||||
|
|
Loading…
Reference in New Issue