-
Notifications
You must be signed in to change notification settings - Fork 64
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
time delays not working correctly #27
Comments
It is because the pykoopman you imported is the one from PyPi, which is a very old version. I just uploaded the 1.0.1 version. You should be able to find the example working now. |
Thanks! I uninstalled and reinstalled, tried both from git clone and pip install pykoopman. Cell In[891], line 1 AttributeError: module 'pykoopman' has no attribute 'regression' I definitely have "regression" folder inside the git cloned folder... EDIT: it works as from pykoopman.regression import DMDc, so I can do that. |
There seems to also be a problem instituting the SVD rank:
gives the error:
It works fine when I don't include the svd_rank parameter. |
can you have a complete self-contained code here? including the data |
please take a look at the cell [5] in https://github.com/dynamicslab/pykoopman/blob/master/docs/tutorial_koopman_hankel_dmdc_for_vdp_system.ipynb So basically, if you are implementing EDMDc with time delay, set In fact, this is not a special case. Here is an example in my JFM paper:
Reference
|
Thanks, that's really helpful! |
It seems like when there is noise in the control signal, it unfortunately leads to overfitting |
tutorial_koopman_hankel_dmdc_for_vdp_system.ipynb
I tried also normal EDMD with time delay as an observable and it errored with the same message.
The text was updated successfully, but these errors were encountered: