Fix cash deduction logic in paper.py#668
Open
lzcndm wants to merge 1 commit into
Open
Conversation
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
1 participant
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.
Pull Request 模板
📋 PR 类型
请标记此 PR 的类型：
📖 PR 描述
变更摘要
修改模拟交易资金扣减逻辑
变更详情
模拟交易买入时资金扣减使用$inc进行原子扣减
相关 Issue
Fixes: #667
🏷️ 标签建议
请为此 PR 建议适当的标签：
enhancement- 新功能或改进bug- Bug 修复documentation- 文档相关refactor- 代码重构performance- 性能优化security- 安全相关llm-adapter- LLM 适配器ui/ux- 用户界面/体验config- 配置相关testing- 测试相关👥 审查者
建议的审查者：
📝 额外说明
感谢您的贡献！ 🎉
请确保您已经阅读并遵循了我们的 贡献指南。如果您有任何问题，请随时在 PR 中提问或联系维护者。