AMReX-Astro / AMReX-Astro/Castro
experiment with the MultiFab version of ParallelFor
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- C++
- Stars
- 340
- Forks
- 105
- Avg merge
- 3d 8h
- Merged PRs (30d)
- 8
Description
There is a version of ParallelFor that an iterate of MultiFabs directly. We should experiment with this in some places where we don't create a lot of temprary FABs.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the existing MultiFab version of ParallelFor and the places where Castro creates temporary FABs. Compare candidate call sites and determine whether direct MultiFab iteration is appropriate without changing results. Done means the selected places have been evaluated or updated and the relevant simulation behavior remains correct.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- hpc, performance
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100