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 针对 DeliveryJobs 的“货物装箱”流程,在“装满货物后下一步”节点中引入新的“装满状态”识别模板,避免因鼠标遮挡导致旧模板识别失败,从而无法继续点击下一步。
Changes:
- 新增“货物已装满”状态的模板图片(分别用于 Win32 与 ADB 资源集)。
- 在
PackCargo.json中新增DeliveryJobsCheckCargoFilledToMax节点,并将“装满货物后下一步”的判定从DeliveryJobsCargoFillToMax切换为新节点。 - 对测试用例 JSON 做了格式化调整(单元素数组改为多行)。
Reviewed changes
Copilot reviewed 2 out of 4 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| tests/DeliveryJobs/test_pack_cargo.json | 测试用例 hits 字段格式化为多行数组,便于阅读与扩展。 |
| assets/resource_adb/image/DeliveryJobs/CargoFilledToMax.png | 新增 ADB 资源集下“货物装满”状态模板图。 |
| assets/resource/pipeline/DeliveryJobs/PackCargo.json | 新增装满状态检测节点,并用于“装满货物后下一步”的组合识别条件。 |
| assets/resource/image/DeliveryJobs/CargoFilledToMax.png | 新增 Win32 资源集下“货物装满”状态模板图。 |
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.
Summary by Sourcery
修复 DeliveryJobs 打包货物流程中的问题:当货物已装满且鼠标光标原本会阻挡下一步交互时,现在允许流程继续进行。
Bug Fixes:
Tests:
Original summary in English
Summary by Sourcery
Fix DeliveryJobs pack cargo flow to allow progressing after cargo is full when the mouse cursor would previously block the next step interaction.
Bug Fixes:
Tests: