fix(AutoStockpile&QuantizedSliding): 修正target offset长宽错误#1905
Merged
Conversation
Contributor
There was a problem hiding this comment.
嘿,我已经审查了你的修改,看起来很棒!
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进后续的评审。
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 修正了 QuantizedSliding 与 AutoStockpile 中使用 target_offset 时的宽高偏移计算错误,避免点击区域因 offset 计算不对而发生偏移。
Changes:
- 将
target_offset从[5, 5, -5, -5]更正为[5, 5, -10, -10](对应x/y内移 5 后,w/h需要缩小 10 才能保持左右/上下对称收缩)。 - 同步更新中英文开发文档中的示例值。
- 同步更新 AutoStockpile 的 pipeline 配置与 go-service 动态覆写逻辑。
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| docs/zh_cn/developers/quantized-sliding.md | 更新中文文档中 target_offset 示例为正确的宽高缩减值 |
| docs/en_us/developers/quantized-sliding.md | 更新英文文档中 target_offset 示例为正确的宽高缩减值 |
| assets/resource/pipeline/AutoStockpile/Entry.json | 修正 AutoStockpile 点击节点使用的 target_offset 配置 |
| agent/go-service/quantizedsliding/overrides.go | 修正 go-service 模板点击覆写中 target_offset 的宽高参数 |
MistEO
approved these changes
Apr 3, 2026
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.
哈哈还有铸币(ocz
Summary by Sourcery
调整量化滑动的目标点击偏移配置,并对相关文档进行对齐更新。
Bug Fixes:
Documentation:
target_offset配置。Original summary in English
Summary by Sourcery
Adjust target click offset configuration for quantized sliding and align related documentation.
Bug Fixes:
Documentation: