enh(reports): isolate reports into a new module

The initialization of the 'figures' config for BIDSLayout is made
robuster too.

Author: oesteban

niworkflows/reports/__init__.py

@@ -0,0 +1,6 @@
+# emacs: -*- mode: python; py-indent-offset: 4; indent-tabs-mode: nil -*-
+# vi: set ft=python sts=4 ts=4 sw=4 et:
+
+from .core import generate_reports
+
+__all__ = ['generate_reports']

niworkflows/viz/reports.py renamed to niworkflows/reports/core.py

@@ -20,7 +20,6 @@
 from nipype.utils.filemanip import copyfile
 
 
-add_config_paths(figures=pkgrf('niworkflows', 'viz/figures.json'))
 PLURAL_SUFFIX = defaultdict(str('s').format, [('echo', 'es')])
 SVG_SNIPPET = """\
 <object class="svg-reportlet" type="image/svg+xml" data="./{0}">
@@ -197,7 +196,7 @@ class Report(object):
     >>> robj.generate_report()
     0
     >>> len((testdir / 'out' / 'niworkflows' / 'sub-01.html').read_text())
-    16988
+    17582
 
     """
 
@@ -207,6 +206,11 @@ def __init__(self, reportlets_dir, out_dir, run_uuid, config=None,
         self.root = reportlets_dir
 
         # Initialize a BIDS layout
+        try:
+            add_config_paths(figures=pkgrf('niworkflows', 'reports/figures.json'))
+        except ValueError as e:
+            if "Configuration 'figures' already exists" != str(e):
+                raise
         self.layout = BIDSLayout(self.root, config='figures', validate=False)
 
         # Initialize structuring elements
@@ -225,7 +229,7 @@ def __init__(self, reportlets_dir, out_dir, run_uuid, config=None,
             self.out_filename = 'sub-{}.html'.format(self.subject_id)
 
         if config is None:
-            config = pkgrf('niworkflows', 'viz/defaultconfig.json')
+            config = pkgrf('niworkflows', 'reports/defaultconfig.json')
         self._load_config(Path(config))
 
     def _load_config(self, config):

niworkflows/viz/defaultconfig.json renamed to niworkflows/reports/defaultconfig.json

@@ -5,3 +5,5 @@
 
 from .plots import plot_carpet
 from .utils import SVGNS
+
+__all__ = ['plot_carpet', 'SVGNS']

niworkflows/viz/figures.json renamed to niworkflows/reports/figures.json

@@ -33,8 +33,9 @@ def main():
         'niworkflows': [
             'data/t1-mni_registration*.json',
             'data/bold-mni_registration*.json',
-            'viz/*.json',
-            'viz/report.tpl',
+            'reports/defaultconfig.json',
+            'reports/figures.json',
+            'reports/report.tpl',
         ]}
 
     root_dir = op.dirname(op.abspath(getfile(currentframe())))