fix: harden neutralization and Baostock data handling#35
Merged
initial-d merged 2 commits intoJul 16, 2026
Conversation
initial-d
force-pushed
the
feat-baostock-fix-review
branch
from
July 16, 2026 15:39
50842a1 to
c2be285
Compare
redamancy231-create
force-pushed
the
feat-baostock-fix-review
branch
from
July 16, 2026 15:40
c2be285 to
7c86a59
Compare
- Merge upstream initial-d#34 + initial-d#33 with review fixes - Keep upstream load_validation_panel validation + docs - Keep 21-test neutralize suite with guard fix - Keep source-aware _select_tickers + VWAP fix Co-Authored-By: Claude <noreply@anthropic.com>
redamancy231-create
force-pushed
the
feat-baostock-fix-review
branch
from
July 16, 2026 15:40
7c86a59 to
43012af
Compare
Owner
|
Thanks for the follow-up fixes. I amended the PR before merge to keep the contribution focused and numerically safe:
CI is green on Python 3.9, 3.10, and 3.11. |
initial-d
added a commit
that referenced
this pull request
Jul 16, 2026
Pairing: harden neutralization and Baostock data handling
Owner
|
Post-merge history correction: the contributor branch changed again while the verified CI run was in progress, so the initial merge briefly referenced an unreviewed duplicate-history commit. The authoritative commits are now:
The resulting tree contains the focused six-file fix described above and no AI-tool trailer. GitHub revoked maintainer write access to the fork branch after merge, so the PR head ref itself could not be rewritten again; this note records the discrepancy explicitly. |
redamancy231-create
added a commit
to redamancy231-create/ml-quant-trading
that referenced
this pull request
Jul 16, 2026
- Rebase on upstream/main after initial-d#35 merge Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
This revision rebases the contribution onto the current
mainand keeps only the new fixes beyond #33 and #34.sh./sz.ticker codes with regression testsThe previously merged Baostock report, dependency declarations, and CLI integration remain unchanged.