astrofrog / astrofrog/fast-histogram
Add OpenMP
- Dominant language
- C
- Stars
- 281
- Forks
- 28
- PR merge metrics
- No merged PRs in 30d
Description
I think it should be possible to rewrite the loops to include OpenMP pragmas. The final update could be protected with `#pragma omp atomic` and you can use `default(shared) private(tx ty ix iy) firstprivate(dataptr)` or something similar. You'll need to redo the loops, however, you'll need for loops instead of the inner while (or maybe the outer do while), and the stride part might need to be precalculated in some way.
Contributor guide
No contributing guide indexed for this repository
Research direction
No implementation file or test is named. Start by locating the C histogram loops and understand their current control flow and stride handling; define completion with a correct OpenMP-enabled implementation whose results match the existing histogram behavior and whose performance can be measured.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- performance
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100