[CodeStyle][Ruff][BUAA][F-[21-27]] Fix ruff FURB diagnostic for 7 files in test/{legacy_test, xpu} and tools/

test/legacy_test/test_dist_fleet_base.py

@@ -384,7 +384,7 @@ def is_listen_failed(logx):
             listen_rgx = "Fail to listen"
 
             with open(logx, "r") as rb:
-                for line in rb.readlines():
+                for line in rb:
                     if listen_rgx in line:
                         is_lf = True
                         break

test/legacy_test/test_hsigmoid_op.py

@@ -27,7 +27,7 @@
 
 
 def find_latest_set(num):
-    return 1 + int(math.floor(math.log(num, 2)))
+    return 1 + int(math.floor(math.log2(num)))
 
 
 class CodeTable:

test/legacy_test/test_imperative_deepcf.py

@@ -193,7 +193,7 @@ def load_data(self):
         num_users = -1
         num_items = -1
         with open(self.data_path, 'r') as f:
-            for l in f.readlines():
+            for l in f:
                 uid, iid, rating = (int(v) for v in l.split('\t'))
                 num_users = max(num_users, uid + 1)
                 num_items = max(num_items, iid + 1)

test/legacy_test/test_nan_inf.py

@@ -87,7 +87,7 @@ def check_op_count(self, log, expected_op_count=None):
                 expected_value,
                 f"The number of operator < {op_type} > is expected to be {expected_value}, but received {actual_value}.",
             )
-        print("")
+        print()
 
     def run_check_nan_inf(self, cmd, expected_op_count=None):
         returncode, out, err = self.run_command(cmd)

test/legacy_test/test_put_along_axis_op.py

@@ -409,11 +409,9 @@ def setUp(self):
         for i in range(5):
             for j in range(5):
                 for k in range(5):
-                    self.target[i, self.index[i, j, k], k] = (
-                        self.value[i, j, k]
-                        if self.value[i, j, k]
-                        < self.target[i, self.index[i, j, k], k]
-                        else self.target[i, self.index[i, j, k], k]
+                    self.target[i, self.index[i, j, k], k] = min(
+                        self.target[i, self.index[i, j, k], k],
+                        self.value[i, j, k],
                     )
         self.inputs = {
             'Input': self.xnp,
@@ -458,11 +456,9 @@ def setUp(self):
         for i in range(5):
             for j in range(5):
                 for k in range(5):
-                    self.target[i, self.index[i, j, k], k] = (
-                        self.value[i, j, k]
-                        if self.value[i, j, k]
-                        < self.target[i, self.index[i, j, k], k]
-                        else self.target[i, self.index[i, j, k], k]
+                    self.target[i, self.index[i, j, k], k] = min(
+                        self.target[i, self.index[i, j, k], k],
+                        self.value[i, j, k],
                     )
         self.inputs = {
             'Input': self.xnp,
@@ -503,11 +499,9 @@ def setUp(self):
         for i in range(5):
             for j in range(5):
                 for k in range(5):
-                    self.target[i, self.index[i, j, k], k] = (
-                        self.value[i, j, k]
-                        if self.value[i, j, k]
-                        > self.target[i, self.index[i, j, k], k]
-                        else self.target[i, self.index[i, j, k], k]
+                    self.target[i, self.index[i, j, k], k] = max(
+                        self.target[i, self.index[i, j, k], k],
+                        self.value[i, j, k],
                     )
         self.inputs = {
             'Input': self.xnp,
@@ -552,11 +546,9 @@ def setUp(self):
         for i in range(5):
             for j in range(5):
                 for k in range(5):
-                    self.target[i, self.index[i, j, k], k] = (
-                        self.value[i, j, k]
-                        if self.value[i, j, k]
-                        > self.target[i, self.index[i, j, k], k]
-                        else self.target[i, self.index[i, j, k], k]
+                    self.target[i, self.index[i, j, k], k] = max(
+                        self.target[i, self.index[i, j, k], k],
+                        self.value[i, j, k],
                     )
         self.inputs = {
             'Input': self.xnp,

test/xpu/test_concat_op_xpu.py

@@ -49,9 +49,7 @@ def setUp(self):
             self.attrs = {'axis': self.axis}
             if self.axis < 0:
                 self.actual_axis = self.axis + len(self.x0.shape)
-                self.actual_axis = (
-                    self.actual_axis if self.actual_axis > 0 else 0
-                )
+                self.actual_axis = max(0, self.actual_axis)
             else:
                 self.actual_axis = self.axis
 

tools/check_ctest_hung.py

@@ -37,7 +37,7 @@ def main():
     started = set()
     passed = set()
     with open(logfile, "r") as fn:
-        for l in fn.readlines():
+        for l in fn:
             if l.find("Test ") != -1 and l.find("Passed") != -1:
                 m = re.search(r"Test\s+#[0-9]*\:\s([a-z0-9_]+)", escape(l))
                 passed.add(m.group(1))