Skip to content

perf: ItemTransfer 添加导航#993

Merged
MistEO merged 7 commits intomainfrom
perf/ItemTransfer
Mar 5, 2026
Merged

perf: ItemTransfer 添加导航#993
MistEO merged 7 commits intomainfrom
perf/ItemTransfer

Conversation

@Daydreamer114
Copy link
Contributor

@Daydreamer114 Daydreamer114 commented Mar 4, 2026 •
edited
Loading

当前问题

  • 照抄了 CharacterControllerExamplePitchDeltaDown 的写法，视角移动目前只有 电脑端-前台 可用，并不符合库存转移任务的预期

Summary by Sourcery

增强内容：

  • 调整 ItemTransfer 接口的管道配置，使其在桌面前台控制之外，亦可支持导航用途。
Original summary in English

Summary by Sourcery

Enhancements:

  • Adjust ItemTransfer interface pipeline configuration to support navigation usage in addition to desktop foreground control.

@Daydreamer114

This comment was marked as outdated.

Sign in to view
@Daydreamer114 Daydreamer114 changed the title Perf/item transfer perf: ItemTransfer 添加导航 Mar 4, 2026
@Daydreamer114
Copy link
Contributor Author

Daydreamer114 commented Mar 4, 2026

嗯……新增节点没用，抬头这一步我还是得写到转移任务内部

@Daydreamer114 Daydreamer114 marked this pull request as ready for review March 4, 2026 18:46
Copilot AI review requested due to automatic review settings March 4, 2026 18:46
sourcery-ai[bot]
sourcery-ai bot reviewed Mar 4, 2026
View reviewed changes
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.

Copilot AI reviewed Mar 4, 2026
View reviewed changes
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

This PR fixes an incorrect ItemTransfer pipeline configuration. The previous approach directly opened the backpack from an arbitrary location, which introduced a semi-transparency rendering issue. The fix forces the game world navigation to the Dijiang reception room (帝江号会客室), raises the camera angle to the maximum before opening the repository menu, and restricts the task to only the Win32-Front controller (since camera movement requires foreground control).

Changes:

  • Navigate to Dijiang reception room (SceneEnterWorldDijiang2) before starting ItemTransfer, replacing the old direct-backpack approach.
  • Add camera pitch adjustment (ItemTransferStartMoveCamera) to raise camera to maximum before opening the repository.
  • Restrict the task controller to Win32-Front only and update the TransferTimes pipeline override from ItemTransferStart to ItemTransferSwitchRepoToOrigin.

Reviewed changes

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

File Description
assets/resource/pipeline/ItemTransfer.json Replaces [JumpBack]SceneEnterMenuBackpackWithDepot with [JumpBack]SceneEnterWorldDijiang2; adds ItemTransferStartMoveCamera and ItemTransferStartOpenRepo nodes for camera and backpack setup; redirects ItemTransferCtrlKeyUpBag loop to ItemTransferSwitchRepoToOrigin
assets/resource_en/pipeline/ItemTransfer.json Removes the now-obsolete ItemTransferStart English OCR override (old "Depot"/"Storage" check), but does not add a new override for the new ItemTransferStart node
assets/tasks/ItemTransfer.json Restricts controller list to Win32-Front only; updates TransferTimes pipeline override target from ItemTransferStart to ItemTransferSwitchRepoToOrigin

@zmdyy0318
Copy link
Contributor

zmdyy0318 commented Mar 5, 2026

如果要无论如何都强制传送一下的话，可以参考下示例

    "SceneExampleTeleportTask": {
        "desc": "传送接口使用示例",
        "pre_delay": 0,
        "post_delay": 0,
        "next": [
            "SceneExampleTeleportTaskInMap",
            "[JumpBack]SceneEnterMapAny"
        ]
    },
    "SceneExampleTeleportTaskInMap": {
        "desc": "判断进入地图菜单，开始传送",
        "recognition": "And",
        "all_of": [
            "InMapAny"
        ],
        "pre_delay": 0,
        "post_delay": 0,
        "next": [
            "SceneExampleTeleportTaskInContinue",
            "[JumpBack]SceneEnterWorldDijiang2"
        ]
    },
    "SceneExampleTeleportTaskInContinue": {
        "desc": "判断进入大世界，传送成功，继续业务流程",
        "recognition": "And",
        "all_of": [
            "InWorld"
        ],
        "pre_delay": 0,
        "post_delay": 0,
        "next": [
            “ItemTransferStart”
        ]
    }

抬头确实不能写在Scene里，先在外面手动打开背包吧，后面等这种场景多了再统一整合

@Daydreamer114
Copy link
Contributor Author

Daydreamer114 commented Mar 5, 2026

嗯嗯，我看如果已经在会客室传送点的光环内，直接抬头就可以，写在任务内部了
除此之外直接调用传送到传送点的接口就能用

我再看看有没有办法搞一下控制器

@MistEO MistEO merged commit a3aa5ba into main Mar 5, 2026
17 checks passed
@MistEO MistEO deleted the perf/ItemTransfer branch March 5, 2026 03:01
Daydreamer114 added a commit that referenced this pull request Mar 5, 2026
@Daydreamer114
Revert "perf: ItemTransfer 添加导航 (#993)"
376485d 
This reverts commit a3aa5ba.
MistEO pushed a commit that referenced this pull request Mar 9, 2026
@Daydreamer114
perf: ItemTransfer 添加导航 (#993)
74fff9f 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
MistEO pushed a commit that referenced this pull request Mar 9, 2026
@Daydreamer114
perf: ItemTransfer 添加导航 (#993)
d4d1854 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sourcery-ai sourcery-ai[bot] sourcery-ai[bot] left review comments

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Daydreamer114 @zmdyy0318 @MistEO