optimize #221

smart-test-ti · Sep 19, 2023 · 3c3c07d · 3c3c07d
1 parent d9964a0
commit 3c3c07d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 40 deletions.
diff --git a/solox/public/common.py b/solox/public/common.py
@@ -385,14 +385,18 @@ def instance_type(self, data):
             return 'int'
         else:
             return 'int'
+
+    def open_file(self, path, mode):
+        with open(path, mode) as f:
+            for line in f:
+                yield line
 
     def readLog(self, scene, filename):
         """Read apmlog file data"""
-        log_data_list = []
-        target_data_list = []
+        log_data_list = list()
+        target_data_list = list()
         if os.path.exists(os.path.join(self.report_dir,scene,filename)):
-            f = open(os.path.join(self.report_dir,scene,filename), "r")
-            lines = f.readlines()
+            lines = self.open_file(os.path.join(self.report_dir,scene,filename), "r")
             for line in lines:
                 if isinstance(line.split('=')[1].strip(), int):
                     log_data_list.append({

diff --git a/solox/templates/analysis.html b/solox/templates/analysis.html
@@ -730,12 +730,6 @@ <h5 class="modal-title">{% if lan == 'cn' %} 结果比对 {% else %} Compare {%
                 target:'cpu',
                 platform:platform
             },
-            beforeSend: function () {
-                SwalLoading('Loading','')
-            },
-            complete: function () {
-                swal.close();
-            },
             success: function (data) {
                 cpu_chart.updateSeries([{
                     name: 'app',
@@ -761,12 +755,6 @@ <h5 class="modal-title">{% if lan == 'cn' %} 结果比对 {% else %} Compare {%
                     target:'battery',
                     platform:platform
                 },
-                beforeSend: function () {
-                    SwalLoading('Loading','')
-                },
-                complete: function () {
-                    swal.close();
-                },
                 success: function (data) {
                     if(platform == 'Android'){
                         battery_chart.updateSeries([{
@@ -809,12 +797,6 @@ <h5 class="modal-title">{% if lan == 'cn' %} 结果比对 {% else %} Compare {%
                     target:'fps',
                     platform:platform
                 },
-                beforeSend: function () {
-                    SwalLoading('Loading','')
-                },
-                complete: function () {
-                    swal.close();
-                },
                 success: function (data) {
                     jank_chart.updateSeries([{
                         name: 'jank',
@@ -839,12 +821,6 @@ <h5 class="modal-title">{% if lan == 'cn' %} 结果比对 {% else %} Compare {%
                     target:'gpu',
                     platform:platform
                 },
-                beforeSend: function () {
-                    SwalLoading('Loading','')
-                },
-                complete: function () {
-                    swal.close();
-                },
                 success: function (data) {
                     gpu_chart.updateSeries([{
                         name: 'gpu',
@@ -868,12 +844,6 @@ <h5 class="modal-title">{% if lan == 'cn' %} 结果比对 {% else %} Compare {%
                 target:'fps',
                 platform:platform
             },
-            beforeSend: function () {
-                SwalLoading('Loading','')
-            },
-            complete: function () {
-                swal.close();
-            },
             success: function (data) {
                 fps_chart.updateSeries([{
                     name: 'fps',
@@ -937,12 +907,6 @@ <h5 class="modal-title">{% if lan == 'cn' %} 结果比对 {% else %} Compare {%
                 target:'flow',
                 platform:platform
             },
-            beforeSend: function () {
-                SwalLoading('Loading','')
-            },
-            complete: function () {
-                swal.close();
-            },
             success: function (data) {
                 network_chart.updateSeries([{
                     name: 'send',