chore: sync QPK constraints with QSL bundle 2026.07.2#178
Conversation
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
Co-Authored-By: Codex <noreply@openai.com>
Pigbibi
force-pushed
the
codex/qpk-constraints-2026072
branch
from
a08647b to
49298bb
Compare
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a08647b9e5
ℹ️ 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".
| quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.gitc81901160c5b31127a27dba1c63944933fb6bf | ||
| us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.gitc0dbac5d47546f46d7c08ca2bf080010aa09695 | ||
| hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.gite8f28d171ed4f509b3fcbb790738fc1a7fbd9 | ||
| cn-equity-strategies @ git+https://github.com/QuantStrategyLab/CnEquityStrategies.git | ||
| crypto-strategies @ git+https://github.com/QuantStrategyLab/CryptoStrategies.gite8f0541372b0a658f5685f256cebde361d2563a |
There was a problem hiding this comment.
When these constraints are consumed by pip (for example the downstream compatibility check in .github/workflows/update-qpk-pin.yml installs strategy packages with -c constraints.txt), the missing @ after .git makes pip treat the SHA as part of the repository URL; I verified us-equity-strategies is cloned from https://github.com/QuantStrategyLab/UsEquityStrategies.gitc0db... instead of ...git@c0db.... The CN entry also lost its ref entirely, so these constraints no longer provide fetchable/pinned package refs and downstream installs will fail or drift.
Useful? React with 👍 / 👎.
| hk-equity-strategies @ git+https://github.com/QuantStrategyLab/HkEquityStrategies.git@a72595ae8b6e6d49250dc9cb77da0275523d94dd | ||
| cn-equity-strategies @ git+https://github.com/QuantStrategyLab/CnEquityStrategies.git@026d5b4479340901d2a5aa5fdc77ea47c05611ce | ||
| crypto-strategies @ git+https://github.com/QuantStrategyLab/CryptoStrategies.git@746440683f63c3f696a12278aff9c7af030700c2 | ||
| quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.gitc81901160c5b31127a27dba1c63944933fb6bf |
There was a problem hiding this comment.
For consumers that combine the canonical QPK_PIN with this live constraints file, this line now points at a different and truncated QPK ref: QPK_PIN still contains 37c81901160c5b31127a27dba1c63944933fb6bf, while the constraint drops the leading 37 and git cat-file cannot resolve c81901160c5b31127a27dba1c63944933fb6bf in this repo. Even after restoring the URL separator, the self-constraint would conflict with the pin mechanism and fail to fetch the intended QPK commit.
Useful? React with 👍 / 👎.
Co-Authored-By: Codex <noreply@openai.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
1 participant
Updates QPK constraints.txt after the QSL 2026.07.2 downstream merge wave.\n\nValidation:\n- git diff --check\n- python3 /Users/lisiyi/Projects/QuantRuntimeSettings/scripts/check_qsl_compat.py --repo-root . --compat-root /Users/lisiyi/Projects/QuantRuntimeSettings --json