MatthewPeterKelly/poker-stats

Add support for multi-threading

Open

#7 opened on Jun 25, 2023

View on GitHub
 (11 comments) (0 reactions) (0 assignees)Rust (1 fork)github user discovery
enhancementgood first issue

Repository metrics

Stars
 (3 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Overview

Monte-Carlo simulation is perfectly suited for parallel processing: it needs to run the same block of code a huge number of times. It looks like Rayon would be a great choice for implementing this feature.

Implementation Notes

I think that most of the library utilities should be good to go for threading, we just need find an efficient way to aggregate the scores. My initial thought would be to have one AggregateScore object per thread, and then periodically merge those back into a shared (Arc) AggregateScore that synchronizes the data across threads.

Contributor guide

Research direction
Implement multi threading using Rayon: consider parallel iterators for independent simulations and use Arc<Mutex> for shared state to aggregate scores.
Tech stack
rust
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
RustRayon
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60