Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: worker self spawn #1247

Merged
merged 7 commits into from
Jun 6, 2024
Merged

fix: worker self spawn #1247

merged 7 commits into from
Jun 6, 2024

Conversation

xusd320
Copy link
Contributor

@xusd320 xusd320 commented Jun 5, 2024

解决 #1246 中的问题 1。
方案:记住已经处理过的 worker chunks, 下次遇到时跳过。

Summary by CodeRabbit

  • 新功能
    • workerHelper.ts 文件现在包含等待特定消息类型和启动 Web Workers 的功能。

This comment has been minimized.

@xusd320 xusd320 changed the title Fix/worker self spawn fix: worker self spawn Jun 5, 2024
@xusd320 xusd320 changed the title fix: worker self spawn [WIP] fix: worker self spawn Jun 5, 2024
@xusd320 xusd320 changed the title [WIP] fix: worker self spawn WIP fix: worker self spawn Jun 5, 2024
@xusd320 xusd320 changed the title WIP fix: worker self spawn fix: worker self spawn Jun 5, 2024
Copy link
Member

@sorrycc sorrycc left a comment

Choose a reason for hiding this comment

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

用例能不依赖 @AntV,提一个最简的例子出来吗?感觉 antv 如果更新,用例可能就失效了。

@xusd320 xusd320 force-pushed the fix/worker-self-spawn branch from 519baa4 to ed112d5 Compare June 5, 2024 11:47
@xusd320 xusd320 force-pushed the fix/worker-self-spawn branch from ed112d5 to a7c7c97 Compare June 5, 2024 11:54
@xusd320
Copy link
Contributor Author

xusd320 commented Jun 5, 2024

用例能不依赖 @AntV,提一个最简的例子出来吗?感觉 antv 如果更新,用例可能就失效了。

改好了

@xusd320 xusd320 requested a review from sorrycc June 5, 2024 14:17
@sorrycc sorrycc merged commit a1d4778 into master Jun 6, 2024
8 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/worker-self-spawn branch June 6, 2024 00:26
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