-
Notifications
You must be signed in to change notification settings - Fork 34
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
[Spinal][Config] Additional features and new configuration for Spinal #630
base: master
Are you sure you want to change the base?
[Spinal][Config] Additional features and new configuration for Spinal #630
Conversation
…rameter can be set to only during torqe off state.
[Spinal][Servo] debug around reading servo from rqt
…_lib/dynamixel/spinal_v2
…ed on its own config.
…of thrust signal transmission(PWM or DSHOT).
Have you test this branch with any multiliinked robot, like Hydrus ? |
@tongtybj |
I understand that theoretically this PR does not effect the spinal-neuron system, but the real word is much stricky than what we predict or image. For a very large modification such as this PR, it is indispensible to test with real platform as much as possible (mini_quadrotor, hydrus, dragon, delta, etc.). |
OK, I got it. |
@tongtybj |
Excellent! |
@tongtybj |
I solved the above problem by using temporary directives. |
What is this
This PR is an integrated PR with #624 and #594 . The structure of the STM32 project was also changed to accommodate all available boards.
Details
This PR provides following modifications:
TODO
I have checked this branch works in Spinal F7, H7, and H7_v2 without problems. So, after the review by contributes, it can be merged immediately.