fix: 发布真实 T+1 回测执行与版本化结果接口 - #2
Open
quantskills-GuoYJ wants to merge 2 commits into
Open
Conversation
Contributor
Author
|
@adiltj 您好，我们正在为 QuantSkills 社区项目补充可验证、可组合的版本化接口。这个 PR 基于您原创的 这些改动会影响实际回测语义，想请您重点确认：
您可以批准、提出修改、只接受部分提交，或者选择暂不合并。原始作者与许可证归属不会改变；未经您确认，我们不会把该项目纳入正式核心 DAG 或作为公共端点发布。如果希望调整方案，请直接在本 PR 留言，我们会按您的意见处理。谢谢！ |
Collaborator
|
好的，批准接受
Sent from Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: quantskills-GuoYJ ***@***.***>
Sent: Thursday, 13 August 2026 09:50:12
To: quantskills/skill-backtest ***@***.***>
Cc: adiltj ***@***.***>; Mention ***@***.***>
Subject: Re: [quantskills/skill-backtest] fix: 发布真实 T+1 回测执行与版本化结果接口 (PR #2)
[https://avatars.githubusercontent.com/u/213890245?s=20&v=4]quantskills-GuoYJ left a comment (quantskills/skill-backtest#2)<#2 (comment)>
@adiltj<https://github.com/adiltj> 您好，我们正在为 QuantSkills 社区项目补充可验证、可组合的版本化接口。这个 PR 基于您原创的 skill-backtest，主要涉及 T+1 成交与执行价格处理、版本化回测结果 Envelope，以及可追溯的 artifact lineage。
这些改动会影响实际回测语义，想请您重点确认：
1. T+1 成交时点和执行价格是否符合原项目设计；
2. 成本、持仓与收益计算边界是否正确；
3. 回测结果字段和 lineage 是否充分、是否存在过度表达；
4. 是否有必须继续兼容的旧行为。
您可以批准、提出修改、只接受部分提交，或者选择暂不合并。原始作者与许可证归属不会改变；未经您确认，我们不会把该项目纳入正式核心 DAG 或作为公共端点发布。如果希望调整方案，请直接在本 PR 留言，我们会按您的意见处理。谢谢！
―
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=BPTRD7C6JQ6O4QTQRYS36PT5JUNFJA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMRXGQ4TOMRRGA4KM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5274972108>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/BPTRD7AQTDPXGR6663LZRTL5JUNFJAVCNFSNUABGKJSXA33TNF2G64TZHMYTENRXGM3DAMJQGA5US43TOVSTWNJRGE4DAMZXHA4DJILWAI>.
Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS<https://github.com/notifications/mobile/ios/BPTRD7HTFQRWJDFP3CUSIZT5JUNFJA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMRXGQ4TOMRRGA4KM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJKTGN5XXIZLSL5UW64Y> and Android<https://github.com/notifications/mobile/android/BPTRD7A65FU62IQKERVRBN35JUNFJA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKMRXGQ4TOMRRGA4KM4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLTGN5XXIZLSL5QW4ZDSN5UWI>. Download it today!
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
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
2 participants
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.
摘要
验证
依赖
Registry PR：quantskills/registry#3
风险/回滚
执行价格/退出语义变化，失败 revert。
不含 PandaAI/SSQuant/其余 96 端点。