Small bug
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 72/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- fortran
- Domain
- backend
Research direction
Start at micro_pumas_utils.F90 line 379 and compare the expression with commit 5434e2b009295889f073ffb0df8649605199b09d. Check whether the denominator should use 2*dcs as described in the issue, then assess whether the change affects numerical results. Done means the calculation matches the intended formula and the result impact is documented.
Written by the indexing model from the issue text.
Description
I'm working on integrating PUMAS into clubb and had ChatGPT audit our implementation from a physics standpoint, and it flagged a problem with this line
Aparently this should be calculation 1 / ( 2 *dcs ).
It looks like the bug was introduced in this commit 5434e2b009295889f073ffb0df8649605199b09d.
I'm not sure if this affects numerical results, but I thought I'd document it here quickly.
- Dominant language
- Fortran
- Stars
- 12
- Forks
- 24
- PR merge metrics
- No merged PRs in 30d
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
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.
More from ESCOMP/PUMAS
-
Difficulty 5/5 Over a week Newbie friendliness 35/100
-
Difficulty 4/5 3-5 days Newbie friendliness 25/100
-
Integrate TAU Microphysics EmulatorMay be free again @nusbaume claimed this 387 days ago, and no pull request is open. Open
-
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
Similar issues
-
Slack progress shows a tool row for message react calls on the embedded runtimePossibly taken @obviyus claimed this today. Openclawsweeper:fix-shape-clear clawsweeper:queueable-fix clawsweeper:source-repro impact:ux-friction issue-rating: 🦞 diamond lobster no-stale P2
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
openclaw/openclaw#166859 · 2 comments · 1 reaction ·
Maintainers usually reply within 1 day
-
backend bug size:sm
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
chrisbenincasa/tunarr#2237 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
bmander/geomsolver#118 ·
Maintainers usually reply within 1 day
-
bug needs-triage
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
uoftblueprint/canada-basketball#32 ·
Maintainers usually reply within 1 day