Skip to content

[codex] Add IBKR HK low-vol dividend switch plan - #132

Merged
Pigbibi merged 5 commits into
mainfrom
feature/live-enable-hk-low-vol-dividend
Jun 2, 2026
Merged

[codex] Add IBKR HK low-vol dividend switch plan#132
Pigbibi merged 5 commits into
mainfrom
feature/live-enable-hk-low-vol-dividend

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Enable IBKR switch-plan support for hk_low_vol_dividend_quality.
  • Update HK strategy dependency to the runtime/evidence-gated HK strategy commit.
  • Add factor snapshot artifact filename hints for snapshot-backed factor contracts.
  • Preserve dry-run-first behavior with required feature snapshot path and manifest placeholders.
  • Carry latest main runtime guard / heartbeat changes into the branch.

Validation

  • .venv/bin/python -m pytest -q tests/test_runtime_config_support.py tests/test_request_handling.py tests/test_strategy_runtime.py -> 99 passed
  • Switch plan smoke confirms hk_low_vol_dividend_quality True feature_snapshot <required> and factor snapshot artifact hints.

Dependency / merge order

Notes

  • This PR does not change production Cloud Run directly.
  • Real order enablement remains blocked until production factor snapshot, manifest, lineage, dry-run preview, notification log, and operator approval evidence pass validation.

@Pigbibi
Pigbibi marked this pull request as ready for review June 2, 2026 20:01
@Pigbibi
Pigbibi merged commit ced5db4 into main Jun 2, 2026
1 check passed
@Pigbibi
Pigbibi deleted the feature/live-enable-hk-low-vol-dividend branch June 2, 2026 20:01

@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: 8ad8ba6e27

ℹ️ 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
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@71141ce9e8a343cec8e2140994071eea66422bc6
quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@v0.7.36
us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@v0.7.50
hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@hk-low-vol-dividend-quality-20260603

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Pin the HK strategy dependency to an immutable ref

Because this now points hk-equity-strategies at a branch name, any later pip install -r requirements.txt or Cloud Run build can pick up whatever commit that branch has moved to rather than the runtime adapter/version validated by this change. The other strategy deps are pinned to tags and the previous HK dep was pinned to a SHA, so for rollout reproducibility this should use an immutable tag or commit SHA once the HK low-vol adapter is selected.

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