Skip to content

refactor: simplify map tracker infer concurrency#1585

Merged
isHarryh merged 1 commit intoMaaEnd:v2from
Aliothmoon:refactor/map-tracker-infer-concurrency
Mar 24, 2026
Merged

refactor: simplify map tracker infer concurrency#1585
isHarryh merged 1 commit intoMaaEnd:v2from
Aliothmoon:refactor/map-tracker-infer-concurrency

Conversation

@Aliothmoon
Copy link
Copy Markdown
Contributor

@Aliothmoon Aliothmoon commented Mar 24, 2026

Summary by Sourcery

增强内容:

  • 重构 MapTrackerInfer.Run,使用通道(channel)进行位置推断,而不是使用等待组(wait group),以实现更清晰、更简洁的并发控制。
Original summary in English

Summary by Sourcery

Enhancements:

  • Refactor MapTrackerInfer.Run to use a channel for location inference instead of a wait group for clearer and more concise concurrency control.

@Aliothmoon Aliothmoon requested review from Copilot and isHarryh and removed request for Copilot March 24, 2026 17:40
Copy link
Copy Markdown
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.

@isHarryh
Copy link
Copy Markdown
Member

LGTM.

@isHarryh isHarryh merged commit fc62bc8 into MaaEnd:v2 Mar 24, 2026
20 checks passed
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