Include dynamic volatility context in Telegram logs#78
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b79a863596
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "blend_gate_reason_rsi_cap": "RSI 超阈值", | ||
| "blend_gate_reason_bollinger_cap": "突破布林上轨", | ||
| "blend_gate_reason_volatility_delever": "{symbol} {window} 日年化波动率 {volatility} 高于 {threshold},SOXL 转向 {redirect_symbol}", | ||
| "blend_gate_reason_volatility_delever_dynamic": "{symbol} {window} 日年化波动率 {volatility} 高于实际阈值 {threshold}({threshold_detail}),SOXL 转向 {redirect_symbol}", |
There was a problem hiding this comment.
Localize nested threshold detail before formatting
When the strategy emits this new dynamic volatility reason in the same structured signal_display format used by the rest of the Telegram renderer, threshold_detail is another translation key such as blend_gate_volatility_threshold_detail_dynamic: .... _localize_structured_text only recursively localizes reason/reasons, so this template inserts that raw key and arguments into the alert instead of the intended human-readable dynamic/fixed threshold text; the new Telegram context is therefore not actually readable for dynamic volatility signals unless threshold_detail is localized before formatting.
Useful? React with 👍 / 👎.
Summary
Verification