Merge pull request #239 from ACRF-Image-X-Institute/report_names

Report names

Author: bwheelz36

mri_distortion_toolkit/Reports.py

@@ -558,7 +558,7 @@ def _get_template(self):
 
     # public methods
 
-    def write_html_report(self, output_folder=None):
+    def write_html_report(self, output_folder=None, report_name=None):
         """
         Generates a html report encompassing available acquisition information, test results, and intercative
         plotly figures
@@ -572,6 +572,11 @@ def write_html_report(self, output_folder=None):
             self.output_folder = Path(output_folder)
         self._set_up_directory_structure()
 
+        if report_name is None:
+            report_name = 'MR_QA_report.html'
+        else:
+            report_name = str(Path(report_name).with_suffix('html'))
+
         # save plots and update jinja_dict
 
         distortion_v_r_save_name = self._unique_name_generator(self.output_folder / 'plots', 'distortion_v_r.html')
@@ -596,7 +601,7 @@ def write_html_report(self, output_folder=None):
         # set up template
         self._jinja_dict['html_theme_loc'] = self._html_theme
         j2_template = self._get_template()
-        report_name = self._unique_name_generator(self.output_folder, 'MR_QA_report.html', rel_path=False)
+        report_name = self._unique_name_generator(self.output_folder, report_name, rel_path=False)
         report_code = os.path.split(report_name)[1]  # for printing
         report_code = os.path.splitext(report_code)[0]
         report_code = report_code.replace('MR_QA_report_','')

mri_distortion_toolkit/utilities.py

@@ -721,7 +721,8 @@ def plot_compressed_MarkerVolumes(MarkerVolumeList, p_max=20, p_min=-20, title=N
 
     fig, axs = plt.subplots(figsize=[8, 8], ncols=1, nrows=1)
     if not isinstance(MarkerVolumeList, list):
-            MarkerVolumeList = [MarkerVolumeList]
+        MarkerVolumeList = list(MarkerVolumeList)
+
     if not projection_direction in ['x', 'y', 'z']:
         warnings.warn(f'projection direction must be "x", "y", or "z", not {projection_direction}.'
                       f'setting to z and continuing')