Fix bug with updating existing field and strategies with no params

diogomatoschaves · Feb 12, 2024 · cf2490f · cf2490f
1 parent ff155e7
commit cf2490f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/data/sources/binance/load/_load.py b/data/sources/binance/load/_load.py
@@ -54,8 +54,9 @@ def save_new_entry_db(model_class, fields, count_updates=True):
         if count_updates:
 
             fields_subset = {key: value for key, value in fields.items() if key in unique_fields}
+            updating_fields = {key: value for key, value in fields.items() if key not in unique_fields}
 
-            rows = model_class.objects.filter(**fields_subset).update(**fields)
+            rows = model_class.objects.filter(**fields_subset).update(**updating_fields)
 
             if rows != 0 and count_updates:
                 new_entry = True

diff --git a/shared/utils/helpers/_helpers.py b/shared/utils/helpers/_helpers.py
@@ -133,7 +133,10 @@ def get_pipeline_max_window(pipeline_id):
 
         max_value_params = 0
         for strategy in strategies:
-            max_value = max([value for param, value in strategy["params"].items()])
+            try:
+                max_value = max([value for param, value in strategy["params"].items()])
+            except ValueError:
+                continue
 
             if max_value > max_value_params:
                 max_value_params = max_value