Rename VIP→Tier, recalculate equity curve on fee tier change
Renamed all "VIP N" to "Tier N" in config, server, and HTML dropdowns. Added recalc_equity_curve() that rebuilds the equity curve with new fee rates. Previously the equity curve was passed through unchanged when switching tiers, so the chart visually stayed identical even though PnL numbers changed. Now each tier produces its own curve. Example OBI historical: Tier 0 equity ends at 141.5, Tier 6 at 149.7 — the chart visibly shifts up as fees drop from 4.5bp to 2.4bp taker.
This commit is contained in:
+34
-1
@@ -200,6 +200,34 @@ async def get_backtest(name: str):
|
||||
return JSONResponse({"error": "not found"}, status_code=404)
|
||||
|
||||
|
||||
|
||||
def recalc_equity_curve(equity_curve, trades, new_fee_rate, fee_model):
|
||||
"""Rebuild equity curve with new fee rates, preserving gross PnL."""
|
||||
if not equity_curve or not trades:
|
||||
return equity_curve
|
||||
fee_deltas = {}
|
||||
cum_delta = 0.0
|
||||
for t in trades:
|
||||
sz = t.get("size", 0)
|
||||
px = t.get("price", 0)
|
||||
old_fee = t.get("fee", 0)
|
||||
new_fee = sz * px * new_fee_rate * 2
|
||||
delta = old_fee - new_fee
|
||||
cum_delta += delta
|
||||
fee_deltas[t.get("time", "")] = cum_delta
|
||||
|
||||
new_curve = []
|
||||
delta_idx = 0
|
||||
trade_times = list(fee_deltas.keys())
|
||||
cum = 0.0
|
||||
for pt in equity_curve:
|
||||
pt_time = pt.get("t", "")
|
||||
while delta_idx < len(trade_times) and trade_times[delta_idx] <= pt_time:
|
||||
cum = fee_deltas[trade_times[delta_idx]]
|
||||
delta_idx += 1
|
||||
new_curve.append({"t": pt_time, "v": round(pt.get("v", 0) + cum, 6)})
|
||||
return new_curve
|
||||
|
||||
@app.get("/api/backtest/{name}/recalc")
|
||||
async def recalc_backtest(name: str, fee_tier: int = 0, staking_tier: str = "none"):
|
||||
"""Recalculate backtest PnL with different fee tier."""
|
||||
@@ -251,7 +279,12 @@ async def recalc_backtest(name: str, fee_tier: int = 0, staking_tier: str = "non
|
||||
"pnl_net_pct": round(new_pnl_pct, 4),
|
||||
"fees_total": round(new_fees, 4),
|
||||
"total_trades": len(new_trades),
|
||||
"equity_curve": data.get("equity_curve", []),
|
||||
"equity_curve": recalc_equity_curve(
|
||||
data.get("equity_curve", []),
|
||||
data.get("trades", []),
|
||||
new_fee_rate,
|
||||
fee_model
|
||||
),
|
||||
"trades": new_trades[-100:],
|
||||
"sharpe": data.get("sharpe", 0),
|
||||
"sortino": data.get("sortino", 0),
|
||||
|
||||
Reference in New Issue
Block a user