Skip to content

fix: 修复信任奖励入口roi问题 - #903

Merged
Windsland52 merged 1 commit into
MAA1999:mainfrom
APPLe-DF:fix-trust-reward-entry-roi
Aug 28, 2026
Merged

fix: 修复信任奖励入口roi问题#903
Windsland52 merged 1 commit into
MAA1999:mainfrom
APPLe-DF:fix-trust-reward-entry-roi

Conversation

@APPLe-DF

@APPLe-DF APPLe-DF commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

关联 Issue / Related Issue

没有关联 Issue。需求来源:实际使用中发现信任奖励流程点击“角色”入口时,EnterCharacter 节点的 OCR 识别区域(ROI)发生偏移,导致无法稳定识别“角色”按钮并进入角色界面。

变更摘要 / Summary

  • 调整 resource/base/pipeline/character.jsonEnterCharacter 节点(进入角色界面)的 OCR ROI:y 由 353 调整为 382,高度由 108 调整为 106
  • 修复信任奖励入口识别区域偏移问题,使“角色”入口能被稳定识别并正确进入角色界面

验证 / Validation

  • pnpm check —— 本地执行通过(Prettier 格式检查、schema 校验、i18n 校验、MaaFW 完整性检查全部通过,无错误)

影响范围 / Impact

  • 受影响 pipeline:resource/base/pipeline/character.jsonCharacter / EnterCharacter 节点(角色信任奖励流程入口)
  • 不涉及 interface.jsontasks/、文档或 release workflow 改动

截图 / 日志 / 说明 / Screenshots / Logs / Notes

  • 涉及识别修复:EnterCharacter 为 OCR 识别,期望文本 ["角", "色"]
  • ROI 修改前:[1078, 353, 132, 108];修改后:[1078, 382, 132, 106]
  • 当前无设备端截图/日志,如需补充识别调试图片可后续提供

检查清单 / Checklist

  • 我已阅读并遵守 CONTRIBUTING.md / I have read and followed CONTRIBUTING.md.
  • PR 范围清晰,没有混入无关格式化或重构 / The PR has one clear scope and does not include unrelated formatting or refactors.
  • 用户可见行为或流程变化已同步文档 / User-visible behavior or workflow changes are documented. (本次为识别区域 Bug 修复,无文档变更)
  • 我没有提交本地缓存、构建产物、调试文件、下载的 runtime 文件或大体积模型文件 / I did not commit local caches, build outputs, debug files, downloaded runtime files, or large model files.

Summary by Sourcery

Bug Fixes:

  • 修复信任奖励流程中“角色”入口 OCR 识别区域偏移导致无法稳定进入角色界面的问题。
Original summary in English

Summary by Sourcery

Bug Fixes:

  • 修复信任奖励流程中“角色”入口 OCR 识别区域偏移导致无法稳定进入角色界面的问题。

Summary by CodeRabbit

  • Bug Fixes
    • 调整角色进入状态下的文字识别区域,提升 OCR 识别的准确性与稳定性。

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

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.

@coderabbitai

coderabbitai Bot commented Aug 28, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: de1f21d5-2728-4b9e-a866-1850c60fb1ee

📥 Commits

Reviewing files that changed from the base of the PR and between 26953a0 and 5e4f2f3.

📒 Files selected for processing (1)
  • resource/base/pipeline/character.json

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


📝 Walkthrough

Walkthrough

EnterCharacter 状态的 OCR 识别 ROI 已调整。x 坐标和宽度保持不变,y 坐标与高度已更新。

Changes

角色 OCR 识别

Layer / File(s) Summary
更新 OCR 识别区域
resource/base/pipeline/character.json
EnterCharacter 的 OCR ROI 的 y 坐标从 353 改为 382,高度从 108 改为 106。x 坐标 1078 和宽度 132 保持不变。

Estimated code review effort: 1(简单)| ~2 分钟

Merge Risk: ⚪ Minimal · up to 5e4f2

This localized ROI adjustment does not introduce an actionable merge-blocking risk; the PR is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确概括了主要变更,即修复信任奖励流程中入口 ROI 的问题。标题简洁且与代码修改一致。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

Some tools did not complete. Review the errors below.

🔧 Biome (2.5.7)
resource/base/pipeline/character.json

File contains syntax errors that prevent linting: Line 174: Expected a property but instead found '// 如果有一天深蓝出了超过"max_hit"个角色,请增大这个值;建议:角色总数+50≤max_hit≤角色总数+100; https://res1999.hu; Line 176: End of file expected; Line 176: End of file expected; Line 176: End of file expected; Line 189: End of file expected; Line 190: End of file expected; Line 190: End of file expected; Line 190: End of file expected; Line 203: End of file expected; Line 204: End of file expected; Line 204: End of file expected; Line 204: End of file expected; Line 205: End of file expected; Line 206: End of file expected; Line 206: End of file expected; Line 206: End of file expected; Line 230: End of file expected; Line 231: End of file expected; Line 231: End of file expected; Line 231: End of file expected; Line 256: End of file expected; Line 257: End of file expected; Line 257: End of file expected; Line 257: End of file expected; Line 277: End of file expected; Line 278: End of file expected; Line 278: End of file expected; Line 278: End of file expected; Line 308: End of file expected


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Windsland52
Windsland52 merged commit ec8006f into MAA1999:main Aug 28, 2026
4 of 5 checks passed
@APPLe-DF
APPLe-DF deleted the fix-trust-reward-entry-roi branch August 28, 2026 15:05
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