Skip to content

Localize mega-cap strategy status name#165

Merged
Pigbibi merged 1 commit into
mainfrom
codex/localize-strategy-status-names
Jun 18, 2026
Merged

Localize mega-cap strategy status name#165
Pigbibi merged 1 commit into
mainfrom
codex/localize-strategy-status-names

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • rename the zh notification label for mega_cap_leader_rotation_top50_balanced to 美股超大盘50强平衡龙头轮动
  • expose display_name_zh in strategy profile status output for settings/switching UIs
  • update shared strategy package pins to the localized display-name builds

Tests

  • python -m py_compile notifications/telegram.py scripts/print_strategy_profile_status.py
  • git diff --check
  • i18n scan: only SOXL/SOXX ticker remains in zh text; en locale has no CJK

Note: full platform tests require pandas through strategy runtime imports; not installed on this VPS.

@Pigbibi Pigbibi merged commit 430dcf6 into main Jun 18, 2026
1 check failed
@Pigbibi Pigbibi deleted the codex/localize-strategy-status-names branch June 18, 2026 07:53

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f9b99f52d6

ℹ️ 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".

Comment thread requirements.txt
gunicorn
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@9d9682533578c7e5a9e02ea80040b7014898e4dd
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@c7a1ae5e804b720f2ab6412451b9e8ee3341ff20
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@2a711adf60b585ca02932bab9ee1bac7ce1df7c6

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Align the shared kit pin with HK dependency

In a clean pip install -r requirements.txt, this new direct URL conflicts with the still-pinned hk-equity-strategies dependency: HkEquityStrategies at 196f16b... declares quant-platform-kit @ ...@9d968253... in its pyproject.toml, while the top-level requirements now request ...@2a711ad.... pip cannot satisfy two different direct references for the same project, so fresh deployments that install the full requirements set will fail before the app starts; update the HK strategy package/pin together with this bump or remove the conflicting transitive pin.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant