Skip to content

Bugfix: Add missing params, change I to I_z, revert tire params to full-scale#167

Open
TeoIlie wants to merge 2 commits intof1tenth:rl_examplefrom
TeoIlie:bugfix/fix-missing-std-model-params
Open

Bugfix: Add missing params, change I to I_z, revert tire params to full-scale#167
TeoIlie wants to merge 2 commits intof1tenth:rl_examplefrom
TeoIlie:bugfix/fix-missing-std-model-params

Conversation

@TeoIlie
Copy link
Copy Markdown

@TeoIlie TeoIlie commented Oct 14, 2025

What

  • Changed I to I_z to standardize, now that the STD and ST param dicts are merged - they have the same meaning, and I_z is more descriptive for "moment of inertia around z-axis"
  • Fixed a couple arctan functions to tan to match the commonroad reference code
  • For now reverting tire params to full-scale, as the model behaves erratically with the new params

Notes

  • The tire parameters still need to be modified for 1/10 scale

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

Successfully merging this pull request may close these issues.

1 participant