fixes in validation

hpc-io · Apr 12, 2024 · b98fdd4 · b98fdd4
1 parent 63237ae
commit b98fdd4
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 19 deletions.
diff --git a/explorer/dxt.py b/explorer/dxt.py
@@ -1033,9 +1033,6 @@ def generate_spatiality_plot(self, file, report, log_type):
 
                         self.generated_files[file_id].append(output_file)
 
-                    else:
-                        self.logger.warning("no data to generate spatiality plots")
-
                 else:
                     self.logger.error(
                         "failed to generate the spatiality plots (error %s)",
@@ -1084,9 +1081,6 @@ def generate_phase_plot(self, file, report, log_type):
 
                         self.generated_files[file_id].append(output_file)
 
-                    else:
-                        self.logger.warning("no data to generate I/O phase plots")
-
                 else:
                     self.logger.error(
                         "failed to generate I/O phase plots (error %s)",
@@ -1131,16 +1125,13 @@ def generate_ost_usage_operation_plot(self, file, report, log_type):
                     if os.path.exists(output_file):
                         self.logger.info("SUCCESS: {}".format(output_file))
                     else:
-                        self.logger.warning("no data to generate interactive plots")
+                        self.logger.warning("no data to generate interactive OST usage operation plots")
 
                         if self.args.browser:
                             webbrowser.open("file://{}".format(output_file), new=2)
 
                         self.generated_files[file_id].append(output_file)
 
-                    else:
-                        self.logger.warning("no data to generate interactive OST usage operation plots")
-
                 else:
                     self.logger.error(
                         "failed to generate interactive OST usage operation plots (error %s)",

diff --git a/explorer/plots/operation.py b/explorer/plots/operation.py
@@ -908,8 +908,8 @@ def determine_legend(fig, column):
         sOutput, sError = s.communicate()
 
     if s is None or s.returncode == 0:
-    	drishti_output = open(file + ".drishti", "w")
-    	drishti_output.write(sOutput.decode())
+        drishti_output = open(file + ".drishti", "w")
+        drishti_output.write(sOutput.decode())
 
         output_doc = BeautifulSoup()
         output_doc.append(output_doc.new_tag("body"))
@@ -918,15 +918,15 @@ def determine_legend(fig, column):
         with open(options["output"], "r") as html_file:
             output_doc.body.extend(BeautifulSoup(html_file.read(), "html.parser").body)
 
-    with open(file + ".darshan.html", "r") as html_file:
-        output_doc.head.extend(BeautifulSoup(html_file.read(), "html.parser").head)
+        with open(file + ".darshan.html", "r") as html_file:
+            output_doc.head.extend(BeautifulSoup(html_file.read(), "html.parser").head)
 
-    with open(file + ".darshan.html", "r") as html_file:
-        output_doc.body.extend(BeautifulSoup(html_file.read(), "html.parser").body)
+        with open(file + ".darshan.html", "r") as html_file:
+            output_doc.body.extend(BeautifulSoup(html_file.read(), "html.parser").body)
 
-        output_doc.style.append(BeautifulSoup("pre { padding-left: 60px;}", "html.parser"))
+            output_doc.style.append(BeautifulSoup("pre { padding-left: 60px;}", "html.parser"))
 
-        with open(options["output"], "w") as output_file:
-            output_file.write(str(output_doc))
+            with open(options["output"], "w") as output_file:
+                output_file.write(str(output_doc))
     else:
         sys.exit(os.EX_SOFTWARE)