ivolatility-backtesting 1.34__tar.gz → 1.35__tar.gz

{ivolatility_backtesting-1.34 → ivolatility_backtesting-1.35}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ivolatility_backtesting
-Version: 1.34
+Version: 1.35
 Summary: A universal backtesting framework for financial strategies using the IVolatility API.
 Author-email: IVolatility <support@ivolatility.com>
 Project-URL: Homepage, https://ivolatility.com

{ivolatility_backtesting-1.34 → ivolatility_backtesting-1.35}/ivolatility_backtesting/ivolatility_backtesting.py

@@ -3656,10 +3656,10 @@ def _api_call_internal(endpoint, cache_config, debug, debug_level, **kwargs):
                 elif debug or cache_config.get('debug', False):
                     print(f"[CACHE] ✓ Cache hit: {endpoint} ({len(cached_data) if hasattr(cached_data, '__len__') else '?'} records)")
                 
-                # ✅ OPTIMIZED: Return DataFrame directly without conversion!
-                # Flag '_is_dataframe' tells caller to skip pd.DataFrame() creation
+                # ✅ CONSISTENT: Always return list format for backward compatibility
+                # This ensures `if not response.get('data')` works correctly in all strategies
                 if isinstance(cached_data, pd.DataFrame):
-                    return {'data': cached_data, 'status': 'success', '_is_dataframe': True}
+                    return {'data': cached_data.to_dict('records'), 'status': 'success'}
                 return cached_data
             
             if debug_level >= 3:
@@ -3773,6 +3773,84 @@ def _api_call_internal(endpoint, cache_config, debug, debug_level, **kwargs):
         return None
 
 
+# ============================================================
+# API RESPONSE HELPERS
+# ============================================================
+def get_api_data(response, as_dataframe=True):
+    """
+    Universal helper to extract data from api_call response.
+    Works with both list and DataFrame formats from cache.
+    
+    Args:
+        response: Response dict from api_call()
+        as_dataframe: If True (default), always return DataFrame. 
+                      If False, return list of dicts.
+    
+    Returns:
+        DataFrame/list or None if no valid data
+    
+    Example:
+        # Get stock data as DataFrame
+        stock_df = get_api_data(api_call('/equities/eod/stock-prices', ...))
+        if stock_df is None:
+            print("No data!")
+        
+        # Get as list of dicts
+        records = get_api_data(response, as_dataframe=False)
+    """
+    if response is None:
+        return None
+    
+    data = response.get('data')
+    if data is None:
+        return None
+    
+    # Handle DataFrame (from cache)
+    if isinstance(data, pd.DataFrame):
+        if data.empty:
+            return None
+        return data if as_dataframe else data.to_dict('records')
+    
+    # Handle list (from API)
+    if not data:  # empty list
+        return None
+    
+    return pd.DataFrame(data) if as_dataframe else data
+
+
+def is_api_response_valid(response):
+    """
+    Check if api_call response contains valid non-empty data.
+    Works with both list and DataFrame formats.
+    
+    Args:
+        response: Response dict from api_call()
+    
+    Returns:
+        bool: True if response contains valid non-empty data
+    
+    Example:
+        response = api_call('/equities/eod/stock-prices', ...)
+        if not is_api_response_valid(response):
+            print("No data available!")
+            return
+        
+        # Safe to use response['data'] now
+        df = pd.DataFrame(response['data'])
+    """
+    if response is None:
+        return False
+    
+    data = response.get('data')
+    if data is None:
+        return False
+    
+    if isinstance(data, pd.DataFrame):
+        return not data.empty
+    
+    return bool(data)
+
+
 # ============================================================
 # OPTIONS DATA HELPERS
 # ============================================================
@@ -8918,11 +8996,7 @@ def preload_data_universal(config, data_requests=None, debug=False):
                 
                 response = api_call(endpoint, cache_config, debug=debuginfo, **params)
                 if response and 'data' in response:
-                    # ✅ OPTIMIZED: Skip DataFrame creation if already DataFrame (from memory cache)
-                    if response.get('_is_dataframe'):
-                        df = response['data']
-                    else:
-                        df = pd.DataFrame(response['data'])
+                    df = pd.DataFrame(response['data'])
                     if len(df) > 0:
                         all_data.append(df)
         
@@ -8983,11 +9057,7 @@ def preload_data_universal(config, data_requests=None, debug=False):
                     
                     response = api_call(endpoint, cache_config, debug=debuginfo, **params)
                     if response and 'data' in response:
-                        # ✅ OPTIMIZED: Skip DataFrame creation if already DataFrame (from memory cache)
-                        if response.get('_is_dataframe'):
-                            df = response['data']  # Already a DataFrame!
-                        else:
-                            df = pd.DataFrame(response['data'])
+                        df = pd.DataFrame(response['data'])
                         if len(df) > 0:
                             # Add cp type if not in response
                             if cp_type and 'type' not in df.columns and 'optionType' not in df.columns:
@@ -9009,11 +9079,7 @@ def preload_data_universal(config, data_requests=None, debug=False):
             params = base_params.copy()
             response = api_call(endpoint, cache_config, debug=debuginfo, **params)
             if response and 'data' in response:
-                # ✅ OPTIMIZED: Skip DataFrame creation if already DataFrame (from memory cache)
-                if response.get('_is_dataframe'):
-                    df = response['data']
-                else:
-                    df = pd.DataFrame(response['data'])
+                df = pd.DataFrame(response['data'])
                 if len(df) > 0:
                     all_data.append(df)
         
@@ -11217,7 +11283,7 @@ class UniversalCacheManager:
 __all__ = [
     'BacktestResults', 'BacktestAnalyzer', 'ResultsReporter',
     'ChartGenerator', 'ResultsExporter', 'run_backtest', 'run_backtest_with_stoploss',
-    'init_api', 'api_call', 'APIHelper', 'APIManager',
+    'init_api', 'api_call', 'get_api_data', 'is_api_response_valid', 'APIHelper', 'APIManager',
     'ResourceMonitor', 'create_progress_bar', 'update_progress', 'format_time',
     'StopLossManager', 'PositionManager', 'StopLossConfig',
     'calculate_stoploss_metrics', 'print_stoploss_section', 'create_stoploss_charts',

{ivolatility_backtesting-1.34 → ivolatility_backtesting-1.35}/ivolatility_backtesting.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ivolatility_backtesting
-Version: 1.34
+Version: 1.35
 Summary: A universal backtesting framework for financial strategies using the IVolatility API.
 Author-email: IVolatility <support@ivolatility.com>
 Project-URL: Homepage, https://ivolatility.com

{ivolatility_backtesting-1.34 → ivolatility_backtesting-1.35}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "ivolatility_backtesting"
-version = "1.34"
+version = "1.35"
 description = "A universal backtesting framework for financial strategies using the IVolatility API."
 readme = "README.md"
 authors = [