FB-008: close_trade mixes price and tick; stale exits are silent (zero-P&L fallback) [silent-wrong]
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 70/100
Hướng nghiên cứu
Start in strategy.py at the cited lines 231 and 245, then trace get_price and the trade fields used by close_trade. Check existing tests for stale prices, fill-forward behavior, and no-price exits. Done means one tick source is used, the exit lag is recorded, and an optional maximum lag is enforced with tests covering the stated cases.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Severity: silent-wrong · Source: BUGS_FASTBT.md (2026-10-09; seen in skill trial 11)
- The price is read at
ctx.tick(strategy.py:231), but the trade is stamped with thetickargument (strategy.py:245). A caller passing a different tick records a price from one time against another. - The lag returned by
get_priceis discarded, so a fill-forward exit of any age is accepted: from yesterday's close, or from a contract with no bars left. - With no price at all, the exit price falls back to the entry price (zero P&L) with only a
logger.warning.
Observed: on the stale 2025-11-27 monthly gap, the fill-forward and live-only variants exit differently.
Workaround: check the lag before closing; pin the ffill-vs-live exit policy.
Fix: use one tick source; store exit_lag on the trade; add an optional max_exit_lag.
- Ngôn ngữ chính
- Python
- Star
- 28
- Fork
- 17
- Merge trung bình
- 8 giờ 50 phút
- Pull request đã merge (30 ngày)
- 2
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của uberdeveloper/fastbt
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
uberdeveloper/fastbt#31 ·
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 52/100
uberdeveloper/fastbt#30 ·
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
uberdeveloper/fastbt#28 ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
uberdeveloper/fastbt#26 ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 56/100
uberdeveloper/fastbt#25 ·
Tất cả issue của uberdeveloper/fastbt
Issue tương tự
-
customer-reported question
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Azure/azure-sdk-for-python#49548 · 1 bình luận · 1 reaction ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
btclib-org/btclib-wallet#409 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Phase 2 PSA Capability & Career Profile
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
bcgov/digital-journeys#2310 ·
-
bug good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
fastmachinelearning/hls4ml#1582 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Sort modelsĐang mởenhancement good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày