Skip to content

fix(EnvironmentMonitoring): 前往任务地点前需要先追踪任务否则无法完成#1040

Merged
he0119 merged 6 commits intomainfrom
fix/EnvironmentMonitoring
Mar 6, 2026
Merged

fix(EnvironmentMonitoring): 前往任务地点前需要先追踪任务否则无法完成#1040
he0119 merged 6 commits intomainfrom
fix/EnvironmentMonitoring

Conversation

@he0119
Copy link
Contributor

@he0119 he0119 commented Mar 6, 2026

fixed #1029

Summary by Sourcery

错误修复:

  • 修复了 EnvironmentMonitoring 任务在未先显式跟踪任务的情况下无法完成的问题。
Original summary in English

Summary by Sourcery

Bug Fixes:

  • Fix EnvironmentMonitoring tasks that could not be completed unless the mission was explicitly tracked first.

由 Sourcery 提供的摘要

调整 EnvironmentMonitoring 流水线配置,使任务在不需要显式任务跟踪的情况下也能完成。

错误修复:

  • 解决了一个问题:在未先显式跟踪的情况下,EnvironmentMonitoring 任务无法完成。

增强内容:

  • 重构 EnvironmentMonitoring 流水线,将 OutskirtsMonitoringTerminal 配置拆分为独立的 JSON 资源,并引入可复用的按钮配置。
Original summary in English

Summary by Sourcery

Adjust EnvironmentMonitoring pipeline configuration to allow missions to complete without requiring explicit mission tracking.

Bug Fixes:

  • Resolve an issue where EnvironmentMonitoring missions could not be completed unless they were explicitly tracked first.

Enhancements:

  • Refactor the EnvironmentMonitoring pipeline by splitting the OutskirtsMonitoringTerminal configuration into dedicated JSON resources and introducing a reusable button configuration.

Copilot AI review requested due to automatic review settings March 6, 2026 10:55
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey - 我已经审查了你的修改,看起来很棒!


Sourcery 对开源项目是免费的——如果你喜欢我们的评审,请考虑分享它们 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进后续的评审。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@he0119 he0119 marked this pull request as draft March 6, 2026 10:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

本 PR 修复了环境监测(EnvironmentMonitoring)功能中的一个 bug:前往任务地点前需要先追踪任务,否则无法完成。主要通过在"接取委托/前往任务"和"前往世界位置"之间插入一个"追踪任务"步骤来解决。同时重构了主循环结构,将前往城郊监测终端的导航改为 SubTask。

Changes:

  • 为 8 种任务类型新增了 GoTo*Mission*TrackOrGoToTrack*GoTo* 的追踪流程节点,以及配套的 OCR 检测节点(Check*GoToMissionTextIn*MissionTrackMissionButton
  • 重构了 EnvironmentMonitoring.json 主循环:将前往终端的导航提取为 SubTask,调整了循环结构和终止条件
  • EnvironmentMonitoringEnterOutskirtsMonitoringTerminal 的识别条件从仅识别按钮改为同时识别按钮和武陵地区建设界面

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
assets/resource/pipeline/EnvironmentMonitoring.json 重构主循环:将导航到终端改为 SubTask,调整循环和终止逻辑
assets/resource/pipeline/EnvironmentMonitoring/Region.json 更新进入终端识别条件,新增 GoToOutskirtsMonitoringTerminal 节点
assets/resource/pipeline/EnvironmentMonitoring/OutskirtsMonitoringTerminal.json 重命名 Accept 节点,新增追踪流程(GoToMission/TrackOrGoTo/Track)和 OCR 检测节点

@he0119 he0119 marked this pull request as ready for review March 6, 2026 11:19
Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你好——我已经审查了你的更改,一切看起来都很棒!


Sourcery 对开源项目是免费的——如果你觉得我们的审查有帮助,欢迎分享 ✨
帮我变得更有用!请在每条评论上点 👍 或 👎,我会根据你的反馈改进后续的审查。
Original comment in English

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@he0119 he0119 merged commit 7f7caa9 into main Mar 6, 2026
10 checks passed
@he0119 he0119 deleted the fix/EnvironmentMonitoring branch March 6, 2026 11:43
MistEO pushed a commit that referenced this pull request Mar 9, 2026
fixed #1029

## Summary by Sourcery

错误修复:
- 修复了 `EnvironmentMonitoring` 任务在未先显式跟踪任务的情况下无法完成的问题。

<details>
<summary>Original summary in English</summary>

## Summary by Sourcery

Bug Fixes:
- Fix EnvironmentMonitoring tasks that could not be completed unless the
mission was explicitly tracked first.

</details>

## 由 Sourcery 提供的摘要

调整 EnvironmentMonitoring 流水线配置,使任务在不需要显式任务跟踪的情况下也能完成。

错误修复:
- 解决了一个问题:在未先显式跟踪的情况下,EnvironmentMonitoring 任务无法完成。

增强内容:
- 重构 EnvironmentMonitoring 流水线,将 OutskirtsMonitoringTerminal 配置拆分为独立的
JSON 资源,并引入可复用的按钮配置。

<details>
<summary>Original summary in English</summary>

## Summary by Sourcery

Adjust EnvironmentMonitoring pipeline configuration to allow missions to
complete without requiring explicit mission tracking.

Bug Fixes:
- Resolve an issue where EnvironmentMonitoring missions could not be
completed unless they were explicitly tracked first.

Enhancements:
- Refactor the EnvironmentMonitoring pipeline by splitting the
OutskirtsMonitoringTerminal configuration into dedicated JSON resources
and introducing a reusable button configuration.

</details>
MistEO pushed a commit that referenced this pull request Mar 9, 2026
fixed #1029

## Summary by Sourcery

错误修复:
- 修复了 `EnvironmentMonitoring` 任务在未先显式跟踪任务的情况下无法完成的问题。

<details>
<summary>Original summary in English</summary>

## Summary by Sourcery

Bug Fixes:
- Fix EnvironmentMonitoring tasks that could not be completed unless the
mission was explicitly tracked first.

</details>

## 由 Sourcery 提供的摘要

调整 EnvironmentMonitoring 流水线配置,使任务在不需要显式任务跟踪的情况下也能完成。

错误修复:
- 解决了一个问题:在未先显式跟踪的情况下,EnvironmentMonitoring 任务无法完成。

增强内容:
- 重构 EnvironmentMonitoring 流水线,将 OutskirtsMonitoringTerminal 配置拆分为独立的
JSON 资源,并引入可复用的按钮配置。

<details>
<summary>Original summary in English</summary>

## Summary by Sourcery

Adjust EnvironmentMonitoring pipeline configuration to allow missions to
complete without requiring explicit mission tracking.

Bug Fixes:
- Resolve an issue where EnvironmentMonitoring missions could not be
completed unless they were explicitly tracked first.

Enhancements:
- Refactor the EnvironmentMonitoring pipeline by splitting the
OutskirtsMonitoringTerminal configuration into dedicated JSON resources
and introducing a reusable button configuration.

</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

拍照任务打开相机后卡住

2 participants