-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Definition of g #91
Comments
I think you are correct regarding g and what I called π. We should change the signature for the observable to be What I am struggling with is that this doesn't fit directly with what we typically see in the Koopman literature. What you are proposing is more general and is what we should do though. In Koopman you typically consider some map What about this? Here all the domains and ranges match up for the composition
|
@ArnoStrouwen after thinking about this more, I don't think the signature should be I also question if the docs need to be so complicated to include the mappings. I probably makes more sense to just explain the function in sentences and with code snippets. |
You can also access The current text is:
But this should be made even clearer, and also incorporated into the docstrings. |
From the docstring:
π = Initial condition space, β = model parameter space
g: π Γ β β ββΏα΅α΅α΅
But this cannot be true since you can calculate expectation at multiple or even infinite time-points.
It is more like:
g: π Γ π Γ β β ββΏα΅α΅α΅ ,
where π is the time-space.
π Γ π is then the solution space, which is how g is implemented in practice.
and possibly also better to call π state space rather than initial condition space.
The text was updated successfully, but these errors were encountered: