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

Provide a clear error if the lengths of wbi_id_list and ROBOT_DOF do not match #38

Open
traversaro opened this issue Dec 22, 2014 · 4 comments

Comments

@traversaro
Copy link
Member

It is a common error and currently the error that you get does not mention the real issue.

@traversaro traversaro changed the title Provide a clear error if the lenght of wbi_id_list and ROBOT_DOF does not match Provide a clear error if the lenght of wbi_id_list and ROBOT_DOF do not match Dec 22, 2014
@DanielePucci
Copy link
Contributor

Why can't we retrieve the ROBOT_DOF from the length of the wbi_id_list?

@jeljaik jeljaik changed the title Provide a clear error if the lenght of wbi_id_list and ROBOT_DOF do not match Provide a clear error if the lengths of wbi_id_list and ROBOT_DOF do not match Dec 22, 2014
@jeljaik
Copy link
Contributor

jeljaik commented Dec 22, 2014

The thing is that ROBOT_DOF and wbi_id_list live in two different worlds. ROBOT_DOF (which was meant to be temporary) is used only in the Matlab workspace to initialize certain variables while wbi_id_list is only parsed within the blocks in the "C++ world". The solution would be having the Matlab parser that I'm currently using also counting the number of DOF. Gimme a couple of hours and I will come back with a solution.

@jeljaik
Copy link
Contributor

jeljaik commented Dec 22, 2014

Ref #26

@DanielePucci
Copy link
Contributor

Any solution for providing a clear error message? @jeljaik @francesco-romano I think that for non-expert users, it's not easy to recognize the dimension mismatch between the length of wbi_id_list and the variable ROBOT_DOF immediately

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants