CICE model_mod refactor
Open
Nobody has claimed this yet.
cice
quad_utils
- Dominant language
- Fortran
- Stars
- 263
- Forks
- 182
- Avg merge
- 11d 12h
- Merged PRs (30d)
- 7
Description
Goals:
- use
get_dart_vector_indexto get state vector offset - use quad_utils module (place to fix interpolation)
- Ncat loops, cat signal - only keep what is needed for the calculation.
- get dimension from state variable, rather than case statement on qty.
- netcdf utilities
Hanging out:
- There is a
@todo DEPRECATEfor use_default_state_variables https://github.com/NCAR/DART/blob/33c310b124666b77db8d3007a72d49b02cbe8a06/models/cice/model_mod.f90#L2531-L2532
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 in models/cice/model_mod.f90, especially the get_dart_vector_index usage and the use_default_state_variables TODO referenced in the issue. Read the quad_utils module and the existing Ncat, state-variable dimension, and netcdf utility code before defining the refactor boundaries. Done means the listed CICE model_mod goals are addressed without retaining unnecessary calculation-only logic.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- fortran
- Domain
- data
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Quiet
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100