[Task] Move out the state vars DOS, DOE, ... from phenology
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 25
- Forks
- 6
- Avg merge
- 2d 23h
- Merged PRs (30d)
- 1
Description
There are some variables like DOS, DOE, DOA, DOM, DOh that are considered as state variables in phenology. in #124, to be able to track different crop types in one Engine instance, we need to track those variables.
Contributor guide
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 by reading the state-variable definitions in src/diffwofost/physical_models/crop/phenology.py around lines 384–388, then review issue #124 for the tracking context. Identify how DOS, DOE, DOA, DOM, and DOh are currently handled and move their state so different crop types can be tracked in one Engine instance.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 58/100