cleanup entry point

nbconvert2.py

@@ -20,27 +20,39 b' import sys'
 import io
 import os
 
-from converters.template import *
 from converters.template import ConverterTemplate
-from converters.html import ConverterHTML
 # From IPython
 
 # All the stuff needed for the configurable things
 from IPython.config.application import Application
 from IPython.config.loader import ConfigFileNotFound
-from IPython.utils.traitlets import List, Unicode, Type, Bool, Dict, CaselessStrEnum
+from IPython.utils.traitlets import Unicode, Bool
 
-from converters.transformers import (ConfigurableTransformers,ExtractFigureTransformer)
+from converters.transformers import (ExtractFigureTransformer)
 
 from converters.jinja_filters import GlobalConfigurable
 
 
 class NbconvertApp(Application):
+    """A basic application to convert ipynb files
 
-    stdout = Bool(True, config=True)
-    write = Bool(False, config=True)
+    """
+
+    stdout = Bool(True, config=True,
+            help="""
+            Wether to print the converted ipynb file to stdout
+            use full do diff files without actually writing a new file
+            """)
 
-    fileext = Unicode('txt', config=True)
+    write = Bool(False, config=True,
+            help="""Shoudl the converted notebook file be written to disk
+            along with potential extracted resources.
+            """
+            )
+
+    fileext = Unicode('txt', config=True,
+            help="""Extension of the file that should be written to disk"""
+            )
 
     aliases = {
             'stdout':'NbconvertApp.stdout',
@@ -50,8 +62,7 b' class NbconvertApp(Application):'
     flags= {}
     flags['no-stdout'] = (
         {'NbconvertApp' : {'stdout' : False}},
-    """the doc for this flag
-
+    """Do not print converted file to stdout, equivalent to --stdout=False
     """
     )
 
@@ -61,35 +72,38 b' class NbconvertApp(Application):'
         # register class here to have help with help all
         self.classes.insert(0,ExtractFigureTransformer)
         self.classes.insert(0,GlobalConfigurable)
-        # ensure those are registerd
 
     def load_config_file(self, profile_name):
+        """load a config file from the config file dir
+
+        profile_name : {string} name of the profile file to load without file extension.
+        """
         try:
             Application.load_config_file(
                 self,
                 profile_name+'.nbcv',
                 path=[os.path.join(os.getcwdu(),'profile')]
             )
+            return True
         except ConfigFileNotFound:
             self.log.warn("Config file for profile '%s' not found, giving up ",profile_name)
-            exit(1)
+            return False
 
 
     def initialize(self, argv=None):
+        """parse command line and load config"""
         self.parse_command_line(argv)
         cl_config = self.config
-        profile_file = sys.argv[1]
-        self.load_config_file(profile_file)
+        profile_file = argv[1]
+        if not self.load_config_file(profile_file):
+            exit(1)
         self.update_config(cl_config)
 
 
 
     def run(self):
-        """Convert a notebook to html in one step"""
-        template_file = (self.extra_args or [None])[0]
-        ipynb_file = (self.extra_args or [None])[1]
-
-        template_file = sys.argv[1]
+        """Convert a notebook in one step"""
+        ipynb_file = (self.extra_args or [None])[2]
 
         C = ConverterTemplate(config=self.config)
 
@@ -104,17 +118,15 b' class NbconvertApp(Application):'
             with io.open(os.path.join(out_root+'.'+self.fileext),'w') as f:
                         f.write(output)
         if keys :
-            if self.write and not os.path.exists(out_root+'_files'):
-                os.mkdir(out_root+'_files')
-            for key in keys:
                 if self.write:
-                    with io.open(os.path.join(out_root+'_files',key),'wb') as f:
-                        print(' writing to ',os.path.join(out_root,key))
+                files_dir = out_root+'_files'
+                if not os.path.exists(out_root+'_files'):
+                    os.mkdir(files_dir)
+                for key in keys:
+                    with io.open(os.path.join(files_dir, key), 'wb') as f:
                         f.write(resources['figures'][key])
-            if self.stdout:
-                print('''
-====================== Keys in Resources ==================================
-''')
+            elif self.stdout:
+                print('''====================== Keys in Resources ==================================''')
                 print(resources['figures'].keys())
                 print("""
 ===========================================================================
@@ -127,7 +139,7 b' def main():'
     """Convert a notebook to html in one step"""
     app = NbconvertApp.instance()
     app.description = __doc__
-    app.initialize()
+    app.initialize(argv=sys.argv)
     app.start()
     app.run()
 #-----------------------------------------------------------------------------