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

ensemble_slice_model_simulation is in order #158

Open
navidcy opened this issue Feb 2, 2022 · 4 comments
Open

ensemble_slice_model_simulation is in order #158

navidcy opened this issue Feb 2, 2022 · 4 comments

Comments

@navidcy
Copy link
Collaborator

navidcy commented Feb 2, 2022

Similarly to ensemble_column_model_simulation,

https://github.com/CliMA/OceanTurbulenceParameterEstimation.jl/blob/f7086e11e82e72f3bb7e533c6553ef9cac86031c/src/EnsembleSimulations.jl#L7

it'll be useful to add an ensemble_slice_model_simulation.

simulation = ensemble_slice_model_simulation(observations;
                                             Nensemble = Nensemble,
                                             architecture = architecture,
                                             tracers = (:b, :e, :c),
                                             free_closures = gent_mcwilliams_diffusivity,
                                             additional_closures = closures::Tuple
                                             )

There is an additional difficulty here: the slice simulations usually need to include more than one closures. That is, in the case of the mesoscale eddy fluxes closure we want to be able to calibrate for the gent_mcwilliams_diffusivity (free_closures) but allow for the model ensembles to also include other closures, same for all ensemble members (the additional_closures).

cc @glwagner

@glwagner
Copy link
Member

glwagner commented Feb 2, 2022

I don't understand what's meant by free_closures?

I think you just want one kwarg, closures. Every member of the ensemble is an identical setup, right?

@navidcy
Copy link
Collaborator Author

navidcy commented Feb 7, 2022

By free_closures I meant those whose parameters we calibrate (following the naming for "free parameters") and by additional closures I meant those which are the same for all particles.

But I didn't pay too much thinking to the name -- there may be better choices.

@glwagner
Copy link
Member

glwagner commented Feb 7, 2022

Oh that's interesting. How about a type Free or Calibrated to distinguish?

closure = (Calibrated(gent_mcwilliams_diffusivity), anisotropic_diffusivity)

@glwagner
Copy link
Member

glwagner commented Feb 7, 2022

kwargs work too for sure

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

No branches or pull requests

2 participants