add FIREFLY_URL environment variable as default, change single quotes

Author: stargaser

examples/filetable.py

@@ -15,34 +15,34 @@ def filetable_to_firefly(ffclient, topdir, pattern, recursive=True):
     ffclient: firefly_client.FireflyClient
         Instance of FireflyClient connected to a Firefly server
 
-    topdir: 'str'
+    topdir: "str"
         pathname for directory to search
 
-    pattern: 'str'
-        filename pattern to search for, e.g. '*.fits'
+    pattern: "str"
+        filename pattern to search for, e.g. "*.fits"
 
     recursive: `bool`
         Search all subdirectories recursively (default=True)
 
     Returns:
     --------
-    tbl_val: 'str'
+    tbl_val: "str"
         Descriptor of table that was uploaded to the Firefly server
 
     metadict: `dict`
         Dictionary of metadata items
 
     """
-    filelist = glob(topdir+'/**/' + pattern, recursive=recursive)
-    metadict = {'datasource': 'path'}
-    with tempfile.NamedTemporaryFile(mode='w+t',
+    filelist = glob(topdir+"/**/" + pattern, recursive=recursive)
+    metadict = {"datasource": "path"}
+    with tempfile.NamedTemporaryFile(mode="w+t",
                                      delete=False,
-                                     suffix='.csv') as fd:
+                                     suffix=".csv") as fd:
         csv_writer = csv.writer(fd)
-        csv_writer.writerow(['number','name','path'])
+        csv_writer.writerow(["number","name","path"])
         for i, path in enumerate(filelist):
             csv_writer.writerow([i, os.path.basename(path),
-                                 'file://'+path])
+                                 "file://"+path])
 
     tbl_val = ffclient.upload_file(fd.name)
     os.remove(fd.name)
@@ -55,16 +55,16 @@ def main():
     parser = argparse.ArgumentParser(description="""
         Display a table of files in a Firefly window
         """)
-    parser.add_argument('topdir', help='top-level directory to search')
-    parser.add_argument('pattern', help='filename pattern for search')
-    parser.add_argument('--norecursion', help='do not recursively search topdir',
-                        action='store_true')
-    parser.add_argument('--firefly_url', help='URL for Firefly server',
-                        default='https://irsa.ipac.caltech.edu/irsaviewer')
-    parser.add_argument('--channel', help='channel name for websocket',
+    parser.add_argument("topdir", help="top-level directory to search")
+    parser.add_argument("pattern", help="filename pattern for search")
+    parser.add_argument("--norecursion", help="do not recursively search topdir",
+                        action="store_true")
+    parser.add_argument("--firefly_url", help="URL for Firefly server",
+                        default=os.getenv("FIREFLY_URL"))
+    parser.add_argument("--channel", help="channel name for websocket",
                         default=None)
-    parser.add_argument('--printurl', help='print browser url instead of' +
-                        ' attempting to launch browser', action='store_true')
+    parser.add_argument("--printurl", help="print browser url instead of" +
+                        " attempting to launch browser", action="store_true")
 
     args = parser.parse_args()
     topdir = args.topdir
@@ -89,7 +89,7 @@ def main():
     r = fc.add_cell(0, 0, 1, 2, "tables", "main")
     fc.show_table(tbl_val, meta=metainfo)
     r = fc.add_cell(0, 1, 1, 2, "tableImageMeta", "image-meta")
-    fc.show_image_metadata(viewer_id=r['cell_id'])
+    fc.show_image_metadata(viewer_id=r["cell_id"])
 
 if __name__ == "__main__":
     main()