Merge pull request #326 from superannotateai/sdk_logging

Added time in the logs.

Author: dshabin

src/superannotate/__init__.py

@@ -220,7 +220,7 @@
             "fileHandler": {
                 "class": "logging.handlers.RotatingFileHandler",
                 "level": "DEBUG",
-                "formatter": "consoleFormatter",
+                "formatter": "fileFormatter",
                 "filename": expanduser(constances.LOG_FILE_LOCATION),
                 "mode": "a",
                 "maxBytes": 5 * 1024 * 1024,
@@ -230,6 +230,9 @@
         "formatters": {
             "consoleFormatter": {
                 "format": "SA-PYTHON-SDK - %(levelname)s - %(message)s",
+            },
+            "fileFormatter": {
+                "format": "SA-PYTHON-SDK - %(levelname)s - %(asctime)s - %(message)s"
             }
         },
         "root": {  # root logger
@@ -239,22 +242,28 @@
     }
 )
 
-local_version = parse(__version__)
-if local_version.is_prerelease:
-    logging.info(constances.PACKAGE_VERSION_INFO_MESSAGE.format(__version__))
-req = requests.get("https://pypi.python.org/pypi/superannotate/json")
-if req.ok:
-    releases = req.json().get("releases", [])
-    pip_version = parse("0")
-    for release in releases:
-        ver = parse(release)
-        if not ver.is_prerelease or local_version.is_prerelease:
-            pip_version = max(pip_version, ver)
-    if pip_version.major > local_version.major:
-        logging.warning(
-            constances.PACKAGE_VERSION_MAJOR_UPGRADE.format(local_version, pip_version)
-        )
-    elif pip_version > local_version:
-        logging.warning(
-            constances.PACKAGE_VERSION_UPGRADE.format(local_version, pip_version)
-        )
+
+def log_version_info():
+    local_version = parse(__version__)
+    if local_version.is_prerelease:
+        logging.info(constances.PACKAGE_VERSION_INFO_MESSAGE.format(__version__))
+    req = requests.get("https://pypi.python.org/pypi/superannotate/json")
+    if req.ok:
+        releases = req.json().get("releases", [])
+        pip_version = parse("0")
+        for release in releases:
+            ver = parse(release)
+            if not ver.is_prerelease or local_version.is_prerelease:
+                pip_version = max(pip_version, ver)
+        if pip_version.major > local_version.major:
+            logging.warning(
+                constances.PACKAGE_VERSION_MAJOR_UPGRADE.format(local_version, pip_version)
+            )
+        elif pip_version > local_version:
+            logging.warning(
+                constances.PACKAGE_VERSION_UPGRADE.format(local_version, pip_version)
+            )
+
+
+log_version_info()
+

src/superannotate/lib/app/input_converters/export_from_sa_conversions.py

@@ -75,6 +75,7 @@ def export_from_sa(args):
             args.output_dir, args.input_dir / "classes" / "classes.json"
         )
     except Exception as e:
+        logger.debug(str(e), exc_info=True)
         _create_classes_mapper(args.input_dir, args.output_dir)
 
     data_set = _load_files(args.input_dir, args.task, args.project_type)

src/superannotate/lib/app/input_converters/sa_conversion.py

@@ -204,6 +204,7 @@ def upgrade_json(input_dir, output_dir):
             converted_files.append(file_name)
             write_to_json(output_dir / file_name, output_json)
         except Exception as e:
+            logger.debug(str(e), exc_info=True)
             failed_files.append(file_name)
 
     return converted_files

src/superannotate/lib/core/usecases/annotations.py

@@ -181,7 +181,8 @@ def _upload_annotation(
             if response.errors:
                 return path, False
             return path, True
-        except Exception as _:
+        except Exception as e:
+            logger.debug(str(e), exc_info=True)
             return path, False
 
     def get_bucket_to_upload(self, ids: List[int]):