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

Fix the ICSC 2023 figure #33

Open
moorepants opened this issue Oct 11, 2024 · 3 comments
Open

Fix the ICSC 2023 figure #33

moorepants opened this issue Oct 11, 2024 · 3 comments

Comments

@moorepants
Copy link
Contributor

With recent updates it no longer gives a stable simulation. It should look like what we had, see: #15 (comment)

@moorepants
Copy link
Contributor Author

I went back to e6db13f and ran it. It gives a stable simulation.

image

I have matching parameters and gains in the current version (other than the tire compression parameters) but the simulation gives different results.

Current commit 2f1d766;

image

@moorepants
Copy link
Contributor Author

It may be that the self aligning moment was incorrect or set to zero in the original abstract plot.

@moorepants
Copy link
Contributor Author

moorepants commented Oct 12, 2024

I am thinking that the differences in the model are enough to change the result. You can tweak the gains a bit in the sim (even just using T7 = 10*u4) and get a similar behavior as the icsc original simulation. So maybe there is nothing to solve here other than picking new control gains to get similar motion as before.

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

No branches or pull requests

1 participant