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

matlab-whole-body-simulator install slblocks in non-namespaced directory #51

Open
traversaro opened this issue May 27, 2021 · 2 comments

Comments

@traversaro
Copy link
Contributor

Note, this is probably my fault and of my requests in some old PRs. : )

I noticed that matlab-whole-body-simulator install slblocks.m in <install_prefix>/mex . While this is good as we don't need to add any more paths in the MATLAB path in the robotology-superbuild, it is not ideal as if somebody copies the structure of mwbs and uses it for its own library, the slblocks of the two libraries will be overwritten.

I just opened this as a note, we can discuss about this at the next meeting @nunoguedelha .

@nunoguedelha
Copy link
Collaborator

Hi @traversaro , indeed I had this problem in mind. In immediate solution would be to move the mwbs_xxx.slx library files and the slblocks.m to a folder +mwbs/simulink, similarly to what is done in the whole-body-controllers library. There are other alternatives we can discuss in the meeting.

@nunoguedelha
Copy link
Collaborator

For the mentioned solution, the implementation would be trivial and would not impact users.

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

No branches or pull requests

2 participants