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

Combine Calcs with output time intervals that are subsets of another Calc into a single Calc #291

Open
spencerahill opened this issue Jun 22, 2018 · 0 comments
Labels
Milestone

Comments

@spencerahill
Copy link
Owner

I am currently calculating the climatological annual cycle and annual mean of a bunch of fields using monthly data. So in my calc_suite_specs dict, I have output_time_intervals=[1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 'ann'].

The data for each of the 12 months is already being calculated in the 'ann' case, before they are all averaged together to create the annual mean. So it would make sense to, before that averaging procedure, dispatch each of the desired monthly averages as well. But this is not what aospy currently does. Instead, this generates 13 different Calcs, with the identical data being loaded 13 different times.

This is similar-to-but-different-than #4, because what I'm advocating for here isn't a CalcSuite-level storage of data, but rather implementing multiple output time intervals within a single Calc. But the logic for assessing whether different Calcs are temporal subsets of one another would, I think, belong in CalcSuite.

@spencerahill spencerahill added this to the before v1.0 milestone Jun 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant