Skip to content

perf!: replace rayon with orx-parallel#308

Closed
KSXGitHub wants to merge 1 commit intomasterfrom
try-orx-parallel
Closed

perf!: replace rayon with orx-parallel#308
KSXGitHub wants to merge 1 commit intomasterfrom
try-orx-parallel

Conversation

@KSXGitHub
Copy link
Copy Markdown
Owner

Resolves #305


TODO:

  • Restore the --threads.

@KSXGitHub
Copy link
Copy Markdown
Owner Author

Experiment Results

Local measurement:

  • On small directory ($THIS_PROJECT/target), the rayon implementation is 20+ times faster than the orx-parallel implementation.
  • On big directory ($HOME), the orx-parallel implementation never stops. It is possible there was a deadlock of some kind.

CI: The orx-parallel implementation never stops.

Conclusion

This change is rejected.

@KSXGitHub KSXGitHub closed this Aug 2, 2025
@KSXGitHub KSXGitHub deleted the try-orx-parallel branch August 2, 2025 10:58
@KSXGitHub KSXGitHub mentioned this pull request Aug 2, 2025
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.

perf: try orx-parallel

1 participant