jihoson/The-Ouroboros
1
0
Fork 0
Code Issues 14 Pull Requests 1 Actions 15 Packages Projects Releases Wiki Activity

fix: take_profit_pct defined in schema but never enforced in trading loop #163

New Issue
Closed
opened 2026-02-20 02:58:32 +09:00 by agentson · 0 comments
agentson commented 2026-02-20 02:58:32 +09:00
Collaborator
Copy Link

문제

StockScenario에 take_profit_pct 필드가 정의되어 있지만, main.py의 HOLD 평가 블록에서 익절 조건을 전혀 체크하지 않는다.

현재 코드 (main.py:383-408):

  • HOLD 판정 후 보유 포지션 확인
  • stop_loss_pct 초과 시 SELL 오버라이드 ✅
  • take_profit_pct 도달 시 SELL 오버라이드 ❌ (없음)

영향

AI가 take_profit_pct: 5.0을 생성해도 시스템이 완전히 무시한다.
익절 타이밍 없이 손절만 있는 비대칭 구조.

개선

HOLD 블록에 take_profit 체크 추가:

if loss_pct >= take_profit_threshold:
    decision = TradeDecision(action='SELL', confidence=90, rationale='Take-profit triggered ...')

테스트

  • take_profit 초과 시 SELL 오버라이드 동작 검증
  • stop_loss와 take_profit 동시 조건 처리 검증
## 문제 StockScenario에 take_profit_pct 필드가 정의되어 있지만, main.py의 HOLD 평가 블록에서 익절 조건을 전혀 체크하지 않는다. 현재 코드 (main.py:383-408): - HOLD 판정 후 보유 포지션 확인 - stop_loss_pct 초과 시 SELL 오버라이드 ✅ - take_profit_pct 도달 시 SELL 오버라이드 ❌ (없음) ## 영향 AI가 take_profit_pct: 5.0을 생성해도 시스템이 완전히 무시한다. 익절 타이밍 없이 손절만 있는 비대칭 구조. ## 개선 HOLD 블록에 take_profit 체크 추가: ```python if loss_pct >= take_profit_threshold: decision = TradeDecision(action='SELL', confidence=90, rationale='Take-profit triggered ...') ``` ## 테스트 - take_profit 초과 시 SELL 오버라이드 동작 검증 - stop_loss와 take_profit 동시 조건 처리 검증
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agentson jihoson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#163
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?