Skip to content

Commit dc86af1

Browse files
committed
CLN: a few remaining .format -> f-strings changes (#854 continued)
1 parent c01619a commit dc86af1

File tree

3 files changed

+6
-13
lines changed

3 files changed

+6
-13
lines changed

larray/core/axis.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2912,8 +2912,7 @@ def to_la_ikey(axis, axis_key):
29122912
if isinstance(axis_key, (int, np.integer, slice, Array)):
29132913
return axis_key
29142914
else:
2915-
assert isinstance(axis_key, (list, np.ndarray)), \
2916-
"unsupported key type: {} ({})".format(axis_key, type(axis_key))
2915+
assert isinstance(axis_key, (list, np.ndarray)), f"unsupported key type: {axis_key} ({type(axis_key)})"
29172916
res_axis = axis.subaxis(axis_key)
29182917
# TODO: for perf reasons, we should bypass creating an actual Array by returning axes and key_data
29192918
# but then we will need to implement a function similar to make_numpy_broadcastable which works on axes

larray/core/session.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -988,7 +988,7 @@ def items(self):
988988
>>> arr1, arr2 = arr1.astype(np.int64), arr2.astype(np.int64)
989989
>>> s = Session([('arr2', arr2), ('arr1', arr1), ('group1', group1), ('axis1', axis1)])
990990
>>> for k, v in s.items():
991-
... print("{}: {}".format(k, v.info if isinstance(v, Array) else repr(v)))
991+
... print(f"{k}: {v.info if isinstance(v, Array) else repr(v)}")
992992
arr2: 4
993993
a [4]: 'a0' 'a1' 'a2' 'a3'
994994
dtype: int64
@@ -1427,8 +1427,7 @@ def summary(self, template=None):
14271427
>>> def print_array(key, array):
14281428
... axes_names = ', '.join(array.axes.display_names)
14291429
... shape = ' x '.join(str(i) for i in array.shape)
1430-
... return "{} -> {} ({})\\n title = {}\\n dtype = {}".format(key, axes_names, shape,
1431-
... array.meta.title, array.dtype)
1430+
... return f"{key} -> {axes_names} ({shape})\\n title = {array.meta.title}\\n dtype = {array.dtype}"
14321431
>>> template = {Axis: "{key} -> {name} [{labels}] ({length})",
14331432
... Group: "{key} -> {name}: {axis_name}{labels} ({length})",
14341433
... Array: print_array,

larray/util/misc.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -656,15 +656,10 @@ def wrapper(*args, **kwargs):
656656
new_arg_value = arg_converter(old_arg_value)
657657
else:
658658
new_arg_value = old_arg_value
659-
msg = "The {old_name}={old_val!r} keyword is deprecated, use {new_name}={new_val!r} instead"\
660-
.format(old_name=old_arg_name, old_val=old_arg_value, new_name=new_arg_name,
661-
new_val=new_arg_value)
662-
663-
warnings.warn(msg, FutureWarning, stacklevel=stacklevel)
659+
warnings.warn(f"The {old_arg_name}={old_arg_value!r} keyword is deprecated, "
660+
f"use {new_arg_name}={new_arg_value!r} instead", FutureWarning, stacklevel=stacklevel)
664661
if new_arg_name in kwargs:
665-
msg = "Can only specify '{old_name}' or '{new_name}', not both"\
666-
.format(old_name=old_arg_name, new_name=new_arg_name)
667-
raise ValueError(msg)
662+
raise ValueError(f"Can only specify '{old_arg_name}' or '{new_arg_name}', not both")
668663
else:
669664
kwargs[new_arg_name] = new_arg_value
670665
return func(*args, **kwargs)

0 commit comments

Comments
 (0)