Fix ec macro (#2689)

* Added default architecture * Added custom workflow * Added fixes * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Improved dicker_push.yml * Handling for error * undid docker changes * Fixed tests
OpenBB-finance · Oct 3, 2022 · 5f2360a · 5f2360a
1 parent 87fac96
commit 5f2360a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/openbb_terminal/helpers_denomination.py b/openbb_terminal/helpers_denomination.py
@@ -31,10 +31,12 @@ def transform(
     def apply(
         df: pd.DataFrame, source: DENOMINATION, target: DENOMINATION
     ) -> pd.DataFrame:
+        multiplier = get_denominations()[source] / get_denominations()[target]
+        df = df.astype(float)
         return df.apply(
             lambda series: series
             if skipPredicate is not None and skipPredicate(series)
-            else series * (get_denominations()[source] / get_denominations()[target]),
+            else series * multiplier,
             axis,
         )
 
@@ -45,7 +47,7 @@ def apply(
         )
 
     if maxValue is None:
-        maxValue = df.max().max()
+        maxValue = df.max(numeric_only=True).max(numeric_only=True)
 
     foundTargetDenomination = get_denomination(maxValue)