-
Notifications
You must be signed in to change notification settings - Fork 0
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
20 adding vibration mode analysis #40
Conversation
parity with main
…s' into 20-adding-vibration-mode-analysis
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you change the feedback, I think it is ready to be added to TCutility
formatting fix
Merge main into 20
…s' into 20-adding-vibration-mode-analysis
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
The documentation is clear and I understand what the code tries to achieve without going into the details
Nice checks if a section is present so that the user knows what's going on
Also good addition of tests
Transition state validation using vibrations