-
Notifications
You must be signed in to change notification settings - Fork 10
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
[BUG] The compute_setpoint_control_signal() is throwing a runtime error when using the DQ_DifferentialDriveRobot #23
Comments
Hello, @marcos-pereira, As you know, the If it also doesn't work in MATLAB, we'll need to think of a fix for all the languages in a future version of the library and that will take longer. Cheers, |
Hi @mmmarinho , Thanks for the note. I tried the same code on MATLAB
Output
Since J is a 8x2 matrix, J'*J results in a 2x2 matrix. The damping factor generated is being generated with an identity matrix 3x3 because the dimension of the configuration space is 3. However, I am not sure if only using an identity matrix 2x2 will fix the problem because of the dimensions of the other matrices in the optimization problem. While the DQ_DifferentialDriveRobot is not working with the QP controller, I will try to use the DQ_HolonomicBase robot model in my application. I have already adapted the holonomic_base_control.m to use the QP controller in MATLAB. I will adapt it to Python. In the case of further testing with the DQ_DifferentialDriveRobot, just let me know. |
Hello, @marcos-pereira, Thank you for the report. As I expected, the controller code does not currently behave well for Please let me discuss this internally with @bvadorno and we'll work on a comprehensive fix for a future release. Your approach seems good for the time being. Thanks again for your time. Cheers, |
Hi @marcos-pereira, A good way to go in this case is to use the Best regards, |
Hello @bvadorno , Thanks for the idea. I will check on that. Best regards, |
Bug description
To Reproduce
Code
Output
Expected behavior
Environment:
Additional context
The text was updated successfully, but these errors were encountered: