Conversation
Contributor
There was a problem hiding this comment.
Hey - 我已经检查过你的修改,看起来非常棒!
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈来改进之后的评审。
Original comment in English
Hey - I've reviewed your changes and they look great!
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Contributor
There was a problem hiding this comment.
Pull request overview
本 PR 旨在修复 AutoStockpile 价格复核(详情页 OCR)被“调度券”图标干扰的问题,通过调整识别区域与引入颜色过滤来提升 OCR 稳定性;同时包含两处 pipeline 字段顺序的小幅整理。
Changes:
- 调整 AutoStockpile 详情页价格 OCR 的 ROI/expected,并新增 ColorMatch 过滤节点以屏蔽调度券图标干扰。
- 整理 SceneCommon 中关闭 ADB 退出弹窗节点的等待/延迟字段位置(语义不变)。
- 调整 CreditShopping 中 RefreshGetCredits 节点字段顺序(语义不变)。
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| assets/resource/pipeline/SceneManager/SceneCommon.json | 重新排列 __ScenePrivateCloseADBExit 的 wait/delay 字段位置,保持行为一致但结构更清晰 |
| assets/resource/pipeline/CreditShopping/Shopping.json | 调整 RefreshGetCredits 的字段顺序(enabled/max_hit) |
| assets/resource/pipeline/AutoStockpile/DecisionLoop.json | 更新价格复核 OCR 参数并新增颜色过滤节点以降低图标噪声对识别的影响 |
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.
close #1825