Neaten logger-init code (#196)

Author: johnkerl

apis/python/examples/uniformizer.py

@@ -51,14 +51,10 @@ def main() -> int:
     args = parser.parse_args()
 
     if args.verbose:
-        # tiledbsc.logging.logger.setLevel(logging.INFO)
-        # logger = logging.getLogger('tiledbsc')
-        # logger.setLevel(logging.INFO)
-        # logging.getLogger("tiledbsc").setLevel(logging.INFO)
-        # Not able to get any of the above to 'stick'. The following sets level for the whole app,
-        # not just the tiledbsc library, but that's an acceptable workaround since this CLI does
-        # nothing except invoke the tiledbsc library.
-        logging.basicConfig(level=logging.INFO)
+        tiledbsc.logging.logger.setLevel(logging.INFO)
+        logger = logging.getLogger("tiledbsc")
+        logger.setLevel(logging.INFO)
+        logger.addHandler(logging.StreamHandler())
 
     uniformizer = Uniformizer(
         atlas_uri=args.atlas_uri,

apis/python/tools/ingestor

@@ -100,14 +100,10 @@ select `relative=True`. (This is the default.)
     write_soco = args.soco
     verbose = not args.quiet
     if verbose:
-        # tiledbsc.logging.logger.setLevel(logging.INFO)
-        # logger = logging.getLogger('tiledbsc')
-        # logger.setLevel(logging.INFO)
-        # logging.getLogger("tiledbsc").setLevel(logging.INFO)
-        # Not able to get any of the above to 'stick'. The following sets level for the whole app,
-        # not just the tiledbsc library, but that's an acceptable workaround since this CLI does
-        # nothing except invoke the tiledbsc library.
-        logging.basicConfig(level=logging.INFO)
+        tiledbsc.logging.logger.setLevel(logging.INFO)
+        logger = logging.getLogger("tiledbsc")
+        logger.setLevel(logging.INFO)
+        logger.addHandler(logging.StreamHandler())
 
     if args.relative is not None:
         relative = args.relative[0]

apis/python/tools/outgestor

@@ -78,14 +78,10 @@ def main():
 
     verbose = not args.quiet
     if verbose:
-        # tiledbsc.logging.logger.setLevel(logging.INFO)
-        # logger = logging.getLogger('tiledbsc')
-        # logger.setLevel(logging.INFO)
-        # logging.getLogger("tiledbsc").setLevel(logging.INFO)
-        # Not able to get any of the above to 'stick'. The following sets level for the whole app,
-        # not just the tiledbsc library, but that's an acceptable workaround since this CLI does
-        # nothing except invoke the tiledbsc library.
-        logging.basicConfig(level=logging.INFO)
+        tiledbsc.logging.logger.setLevel(logging.INFO)
+        logger = logging.getLogger("tiledbsc")
+        logger.setLevel(logging.INFO)
+        logger.addHandler(logging.StreamHandler())
 
     soma = tiledbsc.SOMA(input_path)
     tiledbsc.io.to_h5ad(soma, output_path)