DOI-USGS / DOI-USGS/streamMetabolizer

store data outside of calc_dDOdt

Open
#226 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Stan
Stars
47
Forks
25
PR merge metrics
No merged PRs in 30d

Description

right now the calc_dDOdt closure includes the day's data. probably faster to create calc_dDOdt once and then pass in different data each time. see also #220

Contributor guide

Open the contributing guide

Research direction

Start by locating the calc_dDOdt closure and reading the related discussion in issue #220. Trace where the closure is created and called, then verify that repeated calculations can use different days' data without recreating it each time.

Written by the indexing model from the issue text.

Assessment

Tech stack
r
Domain
data
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.