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

Review the calculation workflow for generalization #232

Open
mgdenno opened this issue Aug 21, 2024 · 0 comments
Open

Review the calculation workflow for generalization #232

mgdenno opened this issue Aug 21, 2024 · 0 comments
Milestone

Comments

@mgdenno
Copy link
Contributor

mgdenno commented Aug 21, 2024

We need an evaluation of the workflow we use to perform calculations on the joined timeseries. Can we generalize it to be more generic and more extensible?

  • Pre "group by and aggregate" (i.e. attributes and user defined fields), but this could also be a step in the processing if there was/is a need.
  • Filter (remove rows from the population)
  • Group by (determine the populations to be evaluated)
  • Aggregate (reduce the rows to those created by the groups and add any aggregates, mean, bias, kge, etc),
    • This will be a Python function and could include an additional filter or sub-aggregates (i.e. calculation of mean daily)
    • This can return a single value of a struct or list. If struct or list it may require unpacking.
  • Finalize (A few things can happen here. Multiple aggregated values can be combined, an aggregate the creates a Struct can be unpacked, what else?
@mgdenno mgdenno added this to the v0.4 Release milestone Aug 22, 2024
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