senofy/autofirmer-expanded
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Simplify min-day amount expression, remove extra variable

Browse Source 
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Senofy
2026-03-09 17:38:04 -05:00
co-authored by Claude Sonnet 4.6
parent 68075059d0
commit 95e7433941
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/trading-logic.ts
+2 -5
View File
@@ -79,11 +79,8 @@ export function computeDailyTarget(
// Floor: must make at least minDayPnL today (or whatever is left if less) // Floor: must make at least minDayPnL today (or whatever is left if less)
const floor = Math.min(effectiveMinDay, remaining); const floor = Math.min(effectiveMinDay, remaining);
// When consistency = 0% (baseAmount = 0), there's no consistency-based upper bound — // 0% consistency → no upper bound, take cappedByFuture; otherwise cap at baseAmount
// target cappedByFuture directly (make as much as possible today, reserve future days). const amount = Math.max(floor, baseAmount > 0 ? Math.min(baseAmount, cappedByFuture) : cappedByFuture);
// When consistency > 0%, treat baseAmount as the consistency cap.
const consistencyCapped = baseAmount > 0 ? Math.min(baseAmount, cappedByFuture) : cappedByFuture;
const amount = Math.max(floor, consistencyCapped);
return { amount: Math.round(amount * 100) / 100, path }; return { amount: Math.round(amount * 100) / 100, path };
} }