Consolidate use of the roots solver which is duplicated in clubb_mf and uwshcu.F90
Maintainers usually reply within 4 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 58/100
- Issue type
- Refactor
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- fortran
- Domain
- hpc
Research direction
Start by reading the duplicated roots-solver implementations in clubb_mf and uwshcu.F90 and compare their inputs, outputs, and call sites. Consolidate them into one shared location usable by both routines, then verify that the model answers remain unchanged.
Written by the indexing model from the issue text.
Description
Issue Type
Code Clean-up
Issue Description
There will be one location for roots which can be used by both routines.
Will this change answers?
No
Will you be implementing this yourself?
Any CAM SE can do this
- Dominant language
- No language data
- Stars
- 93
- Forks
- 186
- Avg merge
- 4d 21h
- Merged PRs (30d)
- 18
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/CAM
-
build-namelist hangs (infinite loop) on #-commented line in user_nl_camPossibly taken @jtruesdal claimed this 19 days ago. Openbug misc tag Must have CESM3
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
ESCOMP/CAM#1701 · 2 comments · 1 assignee ·
Maintainers usually reply within 4 days
-
CoupledEval3 Must have CESM3
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 4 days
-
Remove for_testing_allow_non_annual_changes from user_nl_clmPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 1/5 Under an hour Newbie friendliness 25/100
Maintainers usually reply within 4 days
-
after CESM3_0 enhancement
Difficulty 4/5 3-5 days Newbie friendliness 45/100
Maintainers usually reply within 4 days
-
Bring in upcoming PUMAS bug fixes (ESCOMP/PUMAS#81); answer-changing for CAM7 defaultsPossibly taken @peverwhee claimed this 5 days ago. Openbug Must have CESM3
Difficulty 5/5 Over a week Newbie friendliness 25/100
ESCOMP/CAM#1732 · 2 assignees ·
Maintainers usually reply within 4 days
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
flux-framework/flux-pmix#152 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
ROCm/rocm-systems#13525 ·
Maintainers usually reply within 2 days
-
Difficulty 2/5 Under an hour Newbie friendliness 80/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day