dcicutils 8.13.3.1b27__py3-none-any.whl → 8.13.3.1b29__py3-none-any.whl

dcicutils/misc_utils.py

@@ -990,6 +990,10 @@ def to_integer(value: str,
                allow_commas: bool = False,
                allow_multiplier_suffix: bool = False,
                fallback: Optional[Union[int, float]] = None) -> Optional[int]:
+    """
+    Converts the given string value to an int or None or the given fallback value if malformed.
+    See comments in to_number for details on the other arguments.
+    """
     return to_number(value, fallback=fallback, as_float=False,
                      allow_commas=allow_commas,
                      allow_multiplier_suffix=allow_multiplier_suffix)
@@ -999,25 +1003,33 @@ def to_float(value: str,
              allow_commas: bool = False,
              allow_multiplier_suffix: bool = False,
              fallback: Optional[Union[int, float]] = None) -> Optional[int]:
+    """
+    Converts the given string value to a float or None or the given fallback value if malformed.
+    See comments in to_number for details on the other arguments.
+    """
     return to_number(value, fallback=fallback, as_float=True,
                      allow_commas=allow_commas,
                      allow_multiplier_suffix=allow_multiplier_suffix)
 
 
-_TO_NUMBER_MULTIPLIER_K = 1000
-_TO_NUMBER_MULTIPLIER_M = 1000 * _TO_NUMBER_MULTIPLIER_K
-_TO_NUMBER_MULTIPLIER_G = 1000 * _TO_NUMBER_MULTIPLIER_M
-_TO_NUMBER_MULTIPLIER_T = 1000 * _TO_NUMBER_MULTIPLIER_G
+_TO_NUMBER_POWER_OF_TEN_FOR_NOTHING = 0
+_TO_NUMBER_POWER_OF_TEN_FOR_K = 3
+_TO_NUMBER_POWER_OF_TEN_FOR_M = 6
+_TO_NUMBER_POWER_OF_TEN_FOR_G = 9
+_TO_NUMBER_POWER_OF_TEN_FOR_T = 12
 
 _TO_NUMBER_MULTIPLIER_SUFFIXES = {
-    "K": _TO_NUMBER_MULTIPLIER_K,
-    "KB": _TO_NUMBER_MULTIPLIER_K,
-    "M": _TO_NUMBER_MULTIPLIER_M,
-    "MB": _TO_NUMBER_MULTIPLIER_M,
-    "G": _TO_NUMBER_MULTIPLIER_G,
-    "GB": _TO_NUMBER_MULTIPLIER_G,
-    "T": _TO_NUMBER_MULTIPLIER_T,
-    "TB": _TO_NUMBER_MULTIPLIER_T
+    "K": _TO_NUMBER_POWER_OF_TEN_FOR_K,
+    "KB": _TO_NUMBER_POWER_OF_TEN_FOR_K,
+    "M": _TO_NUMBER_POWER_OF_TEN_FOR_M,
+    "MB": _TO_NUMBER_POWER_OF_TEN_FOR_M,
+    "G": _TO_NUMBER_POWER_OF_TEN_FOR_G,
+    "GB": _TO_NUMBER_POWER_OF_TEN_FOR_G,
+    "T": _TO_NUMBER_POWER_OF_TEN_FOR_T,
+    "TB": _TO_NUMBER_POWER_OF_TEN_FOR_T,
+    # B means bytes or bases and BP means base pairs; needs to be last.
+    "B": _TO_NUMBER_POWER_OF_TEN_FOR_NOTHING,
+    "BP": _TO_NUMBER_POWER_OF_TEN_FOR_NOTHING
 }
 
 
@@ -1028,7 +1040,7 @@ def to_number(value: str,
               fallback: Optional[Union[int, float]] = None) -> Optional[Union[int, float]]:
     """
     Converts the given string value to an int, or float if as_float is True,
-    or None or the give fallback value if malformed.
+    or None or the given fallback value if malformed.
 
     If allow_commas is True then allows appropriately placed commas (i.e. every three digits).
 
@@ -1050,7 +1062,7 @@ def to_number(value: str,
         else:
             return value if isinstance(value, int) else fallback
 
-    value_multiplier = 1
+    value_multiplier = _TO_NUMBER_POWER_OF_TEN_FOR_NOTHING
     value_fraction = None
     value_negative = False
 
@@ -1066,29 +1078,27 @@ def to_number(value: str,
         value_upper = value.upper()
         for suffix in _TO_NUMBER_MULTIPLIER_SUFFIXES:
             if value_upper.endswith(suffix):
-                value_multiplier *= _TO_NUMBER_MULTIPLIER_SUFFIXES[suffix]
+                value_multiplier = _TO_NUMBER_MULTIPLIER_SUFFIXES[suffix]
                 if not (value := value[:-len(suffix)].strip()):
                     return fallback
                 break
 
     if (allow_multiplier_suffix is True) or (as_float is True):
-        # Allow for example "1.5K" to mean 1500 (integer).
+        # Allow for example "1.5K" to mean 1500 (int).
         if (dot_index := value.rfind(".")) >= 0:
             if value_fraction := value[dot_index + 1:].strip():
-                try:
-                    value_fraction = float(f"0.{value_fraction}")
-                except Exception:
+                if not value_fraction.isdigit():
                     return fallback
             if not (value := value[:dot_index].strip()):
                 if not value_fraction:
                     return fallback
                 value = "0"
     elif (as_float is not True) and (value_dot_zero_suffix := re.search(r"\.0*$", value)):
-        # Allow for example "123.00" to mean 123 (integer).
+        # Allow for example "123.00" to mean 123 (int).
         value = value[:value_dot_zero_suffix.start()]
 
     if (allow_commas is True) and ("," in value):
-        # Make sure that any commas are properly placed.
+        # Make sure any commas are properly placed/spaced.
         if not re.fullmatch(r"(-?\d{1,3}(,\d{3})*)", value):
             return fallback
         value = value.replace(",", "")
@@ -1096,31 +1106,38 @@ def to_number(value: str,
     if not value.isdigit():
         return fallback
 
-    value = float(value) if as_float is True else int(value)
-
-    if value_fraction:
-        value_float = float(value) + value_fraction
-        # Here we do NOT simply do: value_float *= float(value_multiplier);
-        # because it introduces obvious FLOATing point precision ERRORs; for example,
-        # to_integer("1.5678K", allow_multiplier_suffix=True) would yield 1567.8000000000002
-        # if we simply did 1.5678 * 1000.0; but doing the multiplication 10 at a time obviates this
-        # idiosyncracy yielding 1567.8; this ASSUMES that the multipliers are simple multiples of 10.
-        while value_multiplier > 1:
-            value_float *= 10
-            value_multiplier /= 10
-        if as_float is True:
-            value = value_float
+    if value_multiplier != _TO_NUMBER_POWER_OF_TEN_FOR_NOTHING:
+        # We do string manipulation for the (power of ten) multiplier and NOT normal multiplicative
+        # arithmetic because this can EASILY yield UNEXPECTED floating point related INACCURACIES.
+        # E.g. to_integer("1.5678K", allow_multiplier_suffix=True) would yield 1567.8000000000002
+        # rather than 1567.8 if (we simply did 1.5678 * 1000.0); also tried multiplying by 10 at
+        # a time, and using Decimal, which obviated some, but not all, of the idiosyncrasies.
+        if value_fraction:
+            for _ in range(value_multiplier):
+                if value_fraction:
+                    value += value_fraction[0]
+                    value_fraction = value_fraction[1:]
+                else:
+                    value += "0"
+            if value_fraction:
+                if as_float is not True:
+                    # Left over fraction for int with multiplier, e.g. "1.2345K" -> 1234.5.
+                    return fallback
+                value = float(f"{value}.{value_fraction}")
+            else:
+                value = float(value) if as_float else int(value)
+        else:
+            value = value + ("0" * value_multiplier)
+            value = float(value) if as_float else int(value)
+    elif as_float is True:
+        if value_fraction:
+            value = float(f"{value}.{value_fraction}")
         else:
-            value = int(value_float)
-            if value_float != value:
-                return fallback
+            value = float(value)
     else:
-        value *= value_multiplier
+        value = int(value)
 
-    if value_negative:
-        value = -value
-
-    return value
+    return -value if value_negative else value
 
 
 def to_boolean(value: str, fallback: Optional[Any]) -> Optional[Any]:

{dcicutils-8.13.3.1b27.dist-info → dcicutils-8.13.3.1b29.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: dcicutils
-Version: 8.13.3.1b27
+Version: 8.13.3.1b29
 Summary: Utility package for interacting with the 4DN Data Portal and other 4DN resources
 Home-page: https://github.com/4dn-dcic/utils
 License: MIT

{dcicutils-8.13.3.1b27.dist-info → dcicutils-8.13.3.1b29.dist-info}/RECORD

@@ -45,7 +45,7 @@ dcicutils/license_policies/park-lab-gpl-pipeline.jsonc,sha256=vLZkwm3Js-kjV44nug
 dcicutils/license_policies/park-lab-pipeline.jsonc,sha256=9qlY0ASy3iUMQlr3gorVcXrSfRHnVGbLhkS427UaRy4,283
 dcicutils/license_utils.py,sha256=2Yxnh1T1iuMe6wluwbvpFO_zYSGPxB4-STAMc-vz-YM,47202
 dcicutils/log_utils.py,sha256=7pWMc6vyrorUZQf-V-M3YC6zrPgNhuV_fzm9xqTPph0,10883
-dcicutils/misc_utils.py,sha256=R3nUtmx4nynmdTfc98w3CV2JKYCPZPRBijets1nWM20,114877
+dcicutils/misc_utils.py,sha256=yoccfuCRlHOky31ljxtMDugrNE4QHqaweGWOWjWHI94,115922
 dcicutils/obfuscation_utils.py,sha256=fo2jOmDRC6xWpYX49u80bVNisqRRoPskFNX3ymFAmjw,5963
 dcicutils/opensearch_utils.py,sha256=V2exmFYW8Xl2_pGFixF4I2Cc549Opwe4PhFi5twC0M8,1017
 dcicutils/portal_object_utils.py,sha256=Az3n1aL-PQkN5gOFE6ZqC2XkYsqiwKlq7-tZggs1QN4,11062
@@ -75,8 +75,8 @@ dcicutils/trace_utils.py,sha256=g8kwV4ebEy5kXW6oOrEAUsurBcCROvwtZqz9fczsGRE,1769
 dcicutils/validation_utils.py,sha256=cMZIU2cY98FYtzK52z5WUYck7urH6JcqOuz9jkXpqzg,14797
 dcicutils/variant_utils.py,sha256=2H9azNx3xAj-MySg-uZ2SFqbWs4kZvf61JnK6b-h4Qw,4343
 dcicutils/zip_utils.py,sha256=_Y9EmL3D2dUZhxucxHvrtmmlbZmK4FpSsHEb7rGSJLU,3265
-dcicutils-8.13.3.1b27.dist-info/LICENSE.txt,sha256=qnwSmfnEWMl5l78VPDEzAmEbLVrRqQvfUQiHT0ehrOo,1102
-dcicutils-8.13.3.1b27.dist-info/METADATA,sha256=w4K_ZFbf-pBM62p35V5NoZS4HQWx0jlimtBQ2z73pQE,3440
-dcicutils-8.13.3.1b27.dist-info/WHEEL,sha256=7Z8_27uaHI_UZAc4Uox4PpBhQ9Y5_modZXWMxtUi4NU,88
-dcicutils-8.13.3.1b27.dist-info/entry_points.txt,sha256=W6kEWdUJk9tQ4myAgpehPdebcwvCAZ7UgB-wyPgDUMg,335
-dcicutils-8.13.3.1b27.dist-info/RECORD,,
+dcicutils-8.13.3.1b29.dist-info/LICENSE.txt,sha256=qnwSmfnEWMl5l78VPDEzAmEbLVrRqQvfUQiHT0ehrOo,1102
+dcicutils-8.13.3.1b29.dist-info/METADATA,sha256=5o-BDPQAEwMHLLNdu8BR_8Fy2BW7HBj_sidWUUhXcy4,3440
+dcicutils-8.13.3.1b29.dist-info/WHEEL,sha256=7Z8_27uaHI_UZAc4Uox4PpBhQ9Y5_modZXWMxtUi4NU,88
+dcicutils-8.13.3.1b29.dist-info/entry_points.txt,sha256=W6kEWdUJk9tQ4myAgpehPdebcwvCAZ7UgB-wyPgDUMg,335
+dcicutils-8.13.3.1b29.dist-info/RECORD,,