removed a useless try catch and made another except more verbose

cbutil/data_points.py

@@ -41,10 +41,6 @@ def data_point_factory(run, *,
         tag_keys = run.keys() - no_tag_keys - field_keys - {time_key}
 
     tags = {key: run[key]for key in tag_keys}
-    time = run[time_key]
-    try:
-        time = time_conversion(time)
-    except ValueError:
-        logger.warn(f"{time} could not be transformed with the current pattern")
-        pass
+    time = time_conversion(run[time_key])
+
     return DataPoint(measurement_name, time, fields=fields, tags=tags)

cbutil/util.py

@@ -5,6 +5,10 @@ from datetime import datetime
 from typing import Union, Tuple
 from pathlib import Path
 
+import logging
+
+logger = logging.getLogger(__file__)
+
 
 def remove_newline(string: str) -> str:
     return string.replace("\n", " ").strip()
@@ -55,7 +59,7 @@ def file_time_to_sec(file_path) -> int:
 def time_conversion(time_stamp, *, pattern="%Y-%m-%d %H:%M:%S"):
     try:
         return int(time_stamp)
-    except ValueError:
-        pass
+    except ValueError as e:
+        logger.exception(e)
 
     return int(datetime.strptime(time_stamp, pattern).timestamp())