trixi-framework / trixi-framework/Trixi.jl
Add `@inbounds`
- Dominant language
- Julia
- Stars
- 731
- Forks
- 166
- Avg merge
- 2d 18h
- Merged PRs (30d)
- 25
Description
Starting from the analysis in #3083, we have seen in #3107 how `@inbounds` gives a speed-up that goes up to 20 or 30%.
I open this issue to keep track of the places where `@inbounds` will be gradually added till its completion.
The first PR implementing the `@inbounds` with related docs is #3208.
### 1D
- [ ] TreeMesh
- [ ] BlockFV
- [ ] DGMulti
- [ ] StructuredMesh
### 2D
- [ ] TreeMesh
- [ ] P4estMesh
- [ ] T8code
- [ ] UnstructuredMesh
- [ ] DGMulti
- [ ] BlockFV
- [ ] StructuredMesh
- [ ] FDSBP
### 3D
- [ ] TreeMesh (#3208)
- [ ] P4estMesh
- [ ] T8code
- [ ] DGMulti
- [ ] BlockFV
- [ ] StructuredMesh
Contributor guide
Research direction
Start with the analysis in #3083 and the implementation and related documentation in #3208. Review the remaining unchecked TreeMesh, BlockFV, DGMulti, StructuredMesh, P4estMesh, T8code, UnstructuredMesh, and FDSBP entries across the 1D–3D checklist. Done means the remaining locations have been addressed and the documentation is updated.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- julia
- Domain
- performance
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100