-
Notifications
You must be signed in to change notification settings - Fork 7
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
Optimize should allow observables as the observed_param
#589
Comments
@anirban-chaudhuri That is, edit the model to include a new observable Something like: qoi = [lambda y: obs_max_qoi(samples, contexts = ["TotalInfected"]), lambda y: obs_max_qoi(samples, context = ["f"])] where the two observables were added to the model AMR using MIRA: model_sidarthe.observables["TotalInfected"] = mira.metamodel.template_model.Observable(
name = "TotalInfected",
expression = sympy.Symbol("Infected") + sympy.Symbol("Diagnosed") + sympy.Symbol("Ailing") + sympy.Symbol("Recognized") + sympy.Symbol("Threatened")
)
model_sidarthe.observables["f"] = mira.metamodel.template_model.Observable(
name = "f",
expression = sympy.Symbol("theta") - 2* sympy.Symbol("epsilon")
) |
Is this addressed now with #602? |
This should be it! |
Seems that this is already addressed. |
Currently, optimize can only use state variables in the QoI, but we should be able to use observables as well.
The text was updated successfully, but these errors were encountered: