Skip to content

Commit 3391a34

Browse files
authored
CLN: fix flake8 C406, C409, and some of C408 (#38078)
1 parent 1fc5efd commit 3391a34

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

51 files changed

+338
-270
lines changed

asv_bench/benchmarks/groupby.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -486,7 +486,7 @@ def setup(self):
486486
tmp2 = (np.random.random(10000) * 10.0).astype(np.float32)
487487
tmp = np.concatenate((tmp1, tmp2))
488488
arr = np.repeat(tmp, 10)
489-
self.df = DataFrame(dict(a=arr, b=arr))
489+
self.df = DataFrame({"a": arr, "b": arr})
490490

491491
def time_sum(self):
492492
self.df.groupby(["a"])["b"].sum()

pandas/conftest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -472,8 +472,8 @@ def index_with_missing(request):
472472
if request.param in ["tuples", "mi-with-dt64tz-level", "multi"]:
473473
# For setting missing values in the top level of MultiIndex
474474
vals = ind.tolist()
475-
vals[0] = tuple([None]) + vals[0][1:]
476-
vals[-1] = tuple([None]) + vals[-1][1:]
475+
vals[0] = (None,) + vals[0][1:]
476+
vals[-1] = (None,) + vals[-1][1:]
477477
return MultiIndex.from_tuples(vals)
478478
else:
479479
vals[0] = None

pandas/core/indexers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ def is_empty_indexer(indexer, arr_value: np.ndarray) -> bool:
105105
return True
106106
if arr_value.ndim == 1:
107107
if not isinstance(indexer, tuple):
108-
indexer = tuple([indexer])
108+
indexer = (indexer,)
109109
return any(isinstance(idx, np.ndarray) and len(idx) == 0 for idx in indexer)
110110
return False
111111

pandas/core/indexes/base.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3558,7 +3558,7 @@ def _reindex_non_unique(self, target):
35583558
cur_labels = self.take(indexer[check]).values
35593559
cur_indexer = ensure_int64(length[check])
35603560

3561-
new_labels = np.empty(tuple([len(indexer)]), dtype=object)
3561+
new_labels = np.empty((len(indexer),), dtype=object)
35623562
new_labels[cur_indexer] = cur_labels
35633563
new_labels[missing_indexer] = missing_labels
35643564

pandas/core/indexing.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1934,7 +1934,7 @@ def _align_series(self, indexer, ser: "Series", multiindex_indexer: bool = False
19341934
to the locations selected by `indexer`
19351935
"""
19361936
if isinstance(indexer, (slice, np.ndarray, list, Index)):
1937-
indexer = tuple([indexer])
1937+
indexer = (indexer,)
19381938

19391939
if isinstance(indexer, tuple):
19401940

@@ -2073,7 +2073,7 @@ def __getitem__(self, key):
20732073

20742074
# we could have a convertible item here (e.g. Timestamp)
20752075
if not is_list_like_indexer(key):
2076-
key = tuple([key])
2076+
key = (key,)
20772077
else:
20782078
raise ValueError("Invalid call for scalar access (getting)!")
20792079

pandas/core/internals/managers.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1550,7 +1550,7 @@ def __init__(
15501550
)
15511551

15521552
self.axes = [axis]
1553-
self.blocks = tuple([block])
1553+
self.blocks = (block,)
15541554

15551555
@classmethod
15561556
def from_blocks(

pandas/core/series.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ def _init_dict(self, data, index=None, dtype=None):
368368
values = na_value_for_dtype(dtype)
369369
keys = index
370370
else:
371-
keys, values = tuple([]), []
371+
keys, values = tuple(), []
372372

373373
# Input is now list-like, so rely on "standard" construction:
374374

pandas/io/stata.py

Lines changed: 33 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -873,30 +873,26 @@ def __init__(self):
873873
(255, np.dtype(np.float64)),
874874
]
875875
)
876-
self.DTYPE_MAP_XML = dict(
877-
[
878-
(32768, np.dtype(np.uint8)), # Keys to GSO
879-
(65526, np.dtype(np.float64)),
880-
(65527, np.dtype(np.float32)),
881-
(65528, np.dtype(np.int32)),
882-
(65529, np.dtype(np.int16)),
883-
(65530, np.dtype(np.int8)),
884-
]
885-
)
876+
self.DTYPE_MAP_XML = {
877+
32768: np.dtype(np.uint8), # Keys to GSO
878+
65526: np.dtype(np.float64),
879+
65527: np.dtype(np.float32),
880+
65528: np.dtype(np.int32),
881+
65529: np.dtype(np.int16),
882+
65530: np.dtype(np.int8),
883+
}
886884
# error: Argument 1 to "list" has incompatible type "str";
887885
# expected "Iterable[int]" [arg-type]
888886
self.TYPE_MAP = list(range(251)) + list("bhlfd") # type: ignore[arg-type]
889-
self.TYPE_MAP_XML = dict(
890-
[
891-
# Not really a Q, unclear how to handle byteswap
892-
(32768, "Q"),
893-
(65526, "d"),
894-
(65527, "f"),
895-
(65528, "l"),
896-
(65529, "h"),
897-
(65530, "b"),
898-
]
899-
)
887+
self.TYPE_MAP_XML = {
888+
# Not really a Q, unclear how to handle byteswap
889+
32768: "Q",
890+
65526: "d",
891+
65527: "f",
892+
65528: "l",
893+
65529: "h",
894+
65530: "b",
895+
}
900896
# NOTE: technically, some of these are wrong. there are more numbers
901897
# that can be represented. it's the 27 ABOVE and BELOW the max listed
902898
# numeric data type in [U] 12.2.2 of the 11.2 manual
@@ -3138,24 +3134,22 @@ def _write_map(self) -> None:
31383134
all blocks have been written.
31393135
"""
31403136
if not self._map:
3141-
self._map = dict(
3142-
(
3143-
("stata_data", 0),
3144-
("map", self.handles.handle.tell()),
3145-
("variable_types", 0),
3146-
("varnames", 0),
3147-
("sortlist", 0),
3148-
("formats", 0),
3149-
("value_label_names", 0),
3150-
("variable_labels", 0),
3151-
("characteristics", 0),
3152-
("data", 0),
3153-
("strls", 0),
3154-
("value_labels", 0),
3155-
("stata_data_close", 0),
3156-
("end-of-file", 0),
3157-
)
3158-
)
3137+
self._map = {
3138+
"stata_data": 0,
3139+
"map": self.handles.handle.tell(),
3140+
"variable_types": 0,
3141+
"varnames": 0,
3142+
"sortlist": 0,
3143+
"formats": 0,
3144+
"value_label_names": 0,
3145+
"variable_labels": 0,
3146+
"characteristics": 0,
3147+
"data": 0,
3148+
"strls": 0,
3149+
"value_labels": 0,
3150+
"stata_data_close": 0,
3151+
"end-of-file": 0,
3152+
}
31593153
# Move to start of map
31603154
self.handles.handle.seek(self._map["map"])
31613155
bio = BytesIO()

pandas/tests/arithmetic/test_datetime64.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1288,8 +1288,8 @@ def test_dt64arr_add_sub_relativedelta_offsets(self, box_with_array):
12881288
("seconds", 2),
12891289
("microseconds", 5),
12901290
]
1291-
for i, kwd in enumerate(relative_kwargs):
1292-
off = DateOffset(**dict([kwd]))
1291+
for i, (unit, value) in enumerate(relative_kwargs):
1292+
off = DateOffset(**{unit: value})
12931293

12941294
expected = DatetimeIndex([x + off for x in vec_items])
12951295
expected = tm.box_expected(expected, box_with_array)

pandas/tests/base/test_conversion.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ def test_iterable_map(self, index_or_series, dtype, rdtype):
109109
s = typ([1], dtype=dtype)
110110
result = s.map(type)[0]
111111
if not isinstance(rdtype, tuple):
112-
rdtype = tuple([rdtype])
112+
rdtype = (rdtype,)
113113
assert result in rdtype
114114

115115
@pytest.mark.parametrize(

0 commit comments

Comments
 (0)