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

Move FieldTimeSeriesCollector to Oceananigans? #2216

Closed
glwagner opened this issue Feb 3, 2022 · 1 comment
Closed

Move FieldTimeSeriesCollector to Oceananigans? #2216

glwagner opened this issue Feb 3, 2022 · 1 comment
Labels
feature 🌟 Something new and shiny

Comments

@glwagner
Copy link
Member

glwagner commented Feb 3, 2022

Over in OceanTurbulenceParameterEstimation we've developed a utility called FieldTimeSeriesCollector that can be used in a callback to "collect" field data on the fly while a simulation is running. This is useful for idealized problems and quick experimentation where saving data to disk is unnecessary:

https://github.com/CliMA/OceanTurbulenceParameterEstimation.jl/blob/a05d1d8959ef40f0f3e7873b32caf4271f102592/src/Observations.jl#L257-L308

It'd probably be nice to move this feature into Oceananigans proper at some point so it's more visible and more people can benefit. And we can tweak the design / interface too. Because "field_time_serieses" (a NamedTuple collection of FieldTimeSeries) doesn't quite roll off the tongue.

cc @navidcy

@glwagner glwagner added the feature 🌟 Something new and shiny label Feb 3, 2022
@glwagner
Copy link
Member Author

I'm closing this issue because I'm judging that it's not of current, timely relevance to Oceananigans development. If you would like to make it a higher priority or if you think the issue was closed in error please feel free to re-open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 🌟 Something new and shiny
Projects
None yet
Development

No branches or pull requests

1 participant