Pull req#5
Open
maxydog1004 wants to merge 3 commits into
Open
Conversation
slh04ninja-cmyk
added a commit
to slh04ninja-cmyk/gzl2
that referenced
this pull request
May 4, 2026
…loud#3 namehamzajazz-cloud#4 namehamzajazz-cloud#5 namehamzajazz-cloud#6 #10 - dashboard: fix t.get() crash on pandas (bug namehamzajazz-cloud#1) - dashboard: remove canal_stats table query, use fallback directly (bug namehamzajazz-cloud#6) - telegram_listener: tracker passed in TradeManager constructor (bug namehamzajazz-cloud#4) - telegram_listener: _report_sent → asyncio.Event for thread safety (bug namehamzajazz-cloud#5) - telegram_listener: improve _get_last_pnl fallback for suffix mismatch (bug namehamzajazz-cloud#3) - signal_parser: TP pattern now accepts emoji/CONFIRM/HIT after number (bug #10)
slh04ninja-cmyk
pushed a commit
to slh04ninja-cmyk/gzl2
that referenced
this pull request
May 14, 2026
… PnL >24h, TP/SL logging, CHANNEL_NUM_MAP, SL validation, TP boundary Bug 1 (🔴): PnL = 0 for trades > 24h - Changed history_deals_get window from 24h to 7 days in _get_last_pnl() and _resolve_order() Bug 2 (🔴): All closures logged as TP_HIT - Added _get_close_reason() using mt5.DEAL_REASON_TP/SL - Now logs TP_HIT only when deal.reason == TP, SL_HIT when SL - Fallback to PnL sign for OTHER reasons (manual close, etc.) Bug 3 (🟡): CHANNEL_NUM_MAP lookup for negative channel IDs - Now stores both with and without dash (-100xxx and 100xxx) Bug 5 (🟡): SL patterns 1-4,6 missing range validation (1000-9999) - Added range check to all SL patterns that were missing it Bug 6 (🟡): TP Pattern 1 too greedy (matches ATP, TPUMP) - Added \b word boundary before TP in pattern 1 Also fixed: removed duplicate trailing code block from previous commit
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.