cholla-hydro / cholla-hydro/cholla
PRINT_INITIAL_STATS in Read_Grid_HDF5 is broken
Open
bug
- Dominant language
- C++
- Stars
- 74
- Forks
- 41
- Avg merge
- 1d 17h
- Merged PRs (30d)
- 3
Description
A (years old) merge commit must have accidentally mixed up code within the PRINT_INTIAL_STATS macro and the MHD macro. The current behavior takes averages of magnetic field values and is nonsensical.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at the Read_Grid_HDF5 entry point and locate the PRINT_INITIAL_STATS and MHD macros. Compare their current code with the surrounding merge-related changes to identify the swapped logic. Done means PRINT_INITIAL_STATS reports meaningful statistics without averaging magnetic-field values incorrectly.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- data, hpc
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 45/100