[codex] align runtime dependency constraints#282
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 994d947079
ℹ️ 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".
| longport==3.0.23 | ||
| quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@d786c1140967f0e96e35599d057f0655e5a9ba25 | ||
| us-equity-strategies @ git+https://github.com/QuantStrategyLab/UsEquityStrategies.git@c57fb20528d40f363ae149ae6e3f6b1a240ec237 | ||
| quant-platform-kit @ git+https://github.com/QuantStrategyLab/QuantPlatformKit.git@b0eacd2fe4884f7f2447b704a232e9a121f396c4 |
There was a problem hiding this comment.
Keep QPK pin compatible with HK strategy dependency
In environments that install this file normally, such as the Docker build's python -m pip install -r requirements.txt -c constraints.txt, this new direct QPK pin conflicts with the still-pinned hk-equity-strategies@700c8b... on line 13: that HK package's pyproject.toml declares quant-platform-kit @ ...@d786c114.... Pip's resolver treats two direct requirements for the same distribution at different refs as incompatible, so image builds and local installs will fail unless the HK strategy pin is advanced to a commit that depends on b0eacd... or installed without dependencies.
Useful? React with 👍 / 👎.
Summary
Test Plan