Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue with flow normalized concentration and flux values when using only one year of data #338

Open
katakagi opened this issue Feb 24, 2022 · 0 comments

Comments

@katakagi
Copy link

Running WRTDS with only one year of data, I expect the predicted flow normalized concentration and flux (columns FNConc and FNFlux) and non-flow normalized predicted concentration and flux (column ConcDay and FluxDay) from from the getDaily() function to be the same, since the normalization is only able to use one year. However, when I run with only one year of data, the FN and non-FN values start to diverge after 2/28 (59th day of the year).

Reviewing the code, I suspect there may be an issue with the bin_Qs() function and how it handles the leap day, or possibly in line 106 of estDailyFromSurfacesJ.R:

When I replace allLogQsReplicated <- allLogQsByDayOfYear[localDaily$Day] with allLogQsReplicated <- allLogQsByDayOfYear[localDaily$i] the FN and non-FN values match the entire year. I'm not sure if bin_Qs() should be updated or the above referenced line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant