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

Integrate Nav2 Stack and Refactor ISMC Controller #1

Merged
merged 4 commits into from
Apr 21, 2024
Merged

Conversation

OgnjenX
Copy link

@OgnjenX OgnjenX commented Mar 23, 2024

Introduce Nav2 stack, drawing inspiration from the implementation in Orca4. Additionally, restructure ISMC controller to align with Nav2 requirements. Previously, the controller transmitted PWM values for individual motors using the 'tcm' matrix for mapping. However, Ardusub expects 6DoF representation input (linear and angular velocities by axis), as outlined in https://www.ardusub.com/developers/rc-input-and-output.html. This adjustment facilitates seamless integration with Nav2.

Introduce Nav2 stack, drawing inspiration from the implementation in Orca4. Additionally, restructure ISMC controller to align with Nav2 requirements. Previously, the controller transmitted PWM values for individual motors using the 'tcm' matrix for mapping. However, Ardusub expects 6DoF representation input (linear and angular velocities by axis), as outlined in https://www.ardusub.com/developers/rc-input-and-output.html. This adjustment facilitates seamless integration with Nav2.
Add 'invertPwm' function in thruster_dynamics. Function will invert PWM value on range of kMinPwm - kMaxPwm, so that previously max value becomes mix, and vice versa.
Examples:

old          new

1100         1900
1101         1899
1102         1898
...
Remove commented code in mission_runner.py and reformat file.
Add configurable 'use_sim_time' for nav2 launch.
@OgnjenX OgnjenX merged commit 9558570 into main Apr 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant