search

HebiRobotics / hebi-matlab-api

Public download and issue tracker of the latest HEBI API for MATLAB
http://docs.hebi.us/tools.html#matlab-api
0 stars 0 forks source link

Better error messages on HRDF parsing. #34

Closed daverolli closed 4 years ago

daverolli commented 4 years ago

In one case, I left off the mass parameter from a <rigid-body> type and it took a while to realize where the bug was, since the error in Matlab just showed:

Java exception occurred:
NullPointerException

Some of the parsing seems to return better error messages, like when multiplying rotation matrices for specifying rotations. We were eventually able to track the source of this error down by looking at the errors returned by the C++ API.

daverolli commented 4 years ago

Running API 1.7

ennerf commented 4 years ago

There is a required = true flag, but maybe that doesn't get enforced properly. I'll look into it. Thanks for reporting it.

edit: apparently there is no default validation and the flag pretty much just serves for validation. see https://stackoverflow.com/a/2669640/3574093 . Easy enough to fix.

ennerf commented 4 years ago

added rigid body does not define required 'mass'. Will be live whenever the next release is out.