search

ami-iit / matlab-whole-body-simulator

A robot simulator running on simulink
BSD 3-Clause "New" or "Revised" License
31 stars 9 forks source link

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

Open traversaro opened 3 years ago

traversaro commented 3 years ago

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 commented 3 years ago

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 commented 3 years ago

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