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

Bug: Older AutoKoopman is More Accurate #70

Closed
EthanJamesLew opened this issue May 21, 2023 · 9 comments
Closed

Bug: Older AutoKoopman is More Accurate #70

EthanJamesLew opened this issue May 21, 2023 · 9 comments
Labels
bug Something isn't working

Comments

@EthanJamesLew
Copy link
Owner

I have been trying to run the Koopman falsification code on a different machine and have noticed quite different results. The main branch is giving different results than the version I have (which I installed straight after the fix you provided regarding the time step). I am attaching two figures highlighting this, with the result after one iteration learning the Vanderpool model. The old version learns a very accurate representation whereas the current main does not. I haven’t been able to locate the source of error yet so figured it was best to flag it with you.

image

image

@EthanJamesLew
Copy link
Owner Author

@Abdu-Hekal

@EthanJamesLew EthanJamesLew added the bug Something isn't working label May 21, 2023
@EthanJamesLew
Copy link
Owner Author

@Abdu-Hekal to provide a timeline: I will look at this tomorrow evening. I am quite swamped in other things atm. Sorry for the delay.

@Abdu-Hekal
Copy link
Collaborator

No worries, working on the older version for now!

@EthanJamesLew
Copy link
Owner Author

CC @KochdumperNiklas

@EthanJamesLew
Copy link
Owner Author

EthanJamesLew commented Jun 7, 2023

@Abdu-Hekal can you help me reproduce your issue? I have attempted to test autokoopman for the case you show, notebook here: https://colab.research.google.com/drive/18UPHoEZ0kTxknfUcwqNJarFcr_cOBtb8?usp=sharing

@Abdu-Hekal
Copy link
Collaborator

@EthanJamesLew I have updated the notebook to reproduce the issue. This issue seems to happen when there is a lesser number of trajectories (I have changed training data to 4 trajectories).

@EthanJamesLew
Copy link
Owner Author

@Abdu-Hekal Awesome! I see this
download (1)

However, the new training data got deleted when starting a new colab runtime. May you please email me the data?

@Abdu-Hekal
Copy link
Collaborator

Abdu-Hekal commented Jun 8, 2023 via email

@EthanJamesLew
Copy link
Owner Author

We resolved this and the falsification now uses the latest autokoopman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants