astrofrog / astrofrog/fast-histogram

Add OpenMP

Open
#32 2 comments 0 reactions 0 assignees View on GitHub
help wanted performance
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.