Skip to content

Ohmythreads prototype - #171

Closed
rsenne wants to merge 1 commit into
part3b-hsmm-forwardfrom
ohmythreads-prototype
Closed

rsenne wants to merge 1 commit into
part3b-hsmm-forwardfrom
ohmythreads-prototype

Conversation

@rsenne

@rsenne rsenne commented Sep 25, 2026

Copy link
Copy Markdown
Collaborator

I prototyped OhMyThreads.jl. To be honest I'm not sure its worth it. We can probably abstract the current code without the new libraries/dep.

Benchmark:
Setup: Julia 1.13.0, 4 threads (julia -t 4), N = 8 states, Normal emissions, (minimum time).
"Very different lengths" = 200 sequences, each 5000 long with probability 0.1, otherwise 100.

Case Implementation forward_backward! viterbi
200 sequences × 500 @threads (current) 4.98 ms 2.93 ms
200 sequences × 500 OhMyThreads DynamicScheduler 5.44 ms 2.94 ms
200 sequences, very different lengths @threads (current) 5.92 ms 3.50 ms
200 sequences, very different lengths OhMyThreads DynamicScheduler 6.07 ms 3.48 ms

@rsenne
rsenne marked this pull request as draft September 25, 2026 21:45
@rsenne
rsenne changed the base branch from main to part3b-hsmm-forward September 25, 2026 21:45
@codecov

codecov Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.62%. Comparing base (c564739) to head (af93467).

Additional details and impacted files
@@                   Coverage Diff                   @@
##           part3b-hsmm-forward     #171      +/-   ##
=======================================================
+ Coverage                97.10%   97.62%   +0.52%     
=======================================================
  Files                       25       25              
  Lines                      897      843      -54     
=======================================================
- Hits                       871      823      -48     
+ Misses                      26       20       -6     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@gdalle

gdalle commented Sep 26, 2026

Copy link
Copy Markdown
Member

Yeah that doesn't seem like it's worth the additional dep. Let's keep things as they are for now! Thanks for giving it a shot

@rsenne rsenne closed this Sep 26, 2026
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