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

Robot Wrapper #108

Closed
jcarpent opened this issue Feb 22, 2016 · 7 comments
Closed

Robot Wrapper #108

jcarpent opened this issue Feb 22, 2016 · 7 comments

Comments

@jcarpent
Copy link
Contributor

Create a robot wrapper similar to the existing python class containing a model and several data store in a vector achieving the interface between (model,data) and the main algorithms.

@aelkhour Do you have some precise requests?

@aelkhour
Copy link
Contributor

  • Not throwing exceptions would be great.
  • Not making explicit dynamic allocations (using new operator) would be great.

If this conflicts with your development roadmap, then no worries.

@jcarpent
Copy link
Contributor Author

Concerning the exceptions, why this precise request?

@aelkhour
Copy link
Contributor

Our whole code base is supposed to be exception-free. I know this is not currently the case in pinocchio, which is why this is not a hard requirement.

@jcarpent
Copy link
Contributor Author

Ok. If you want you can just use the instruction -fno-exceptions with gcc.
I think for lambda user, exceptions are mandatory that why we try to populate our code with them.

@aelkhour
Copy link
Contributor

Noted, thanks.

@nmansard
Copy link
Contributor

It indeed is: please no throw in construction and destruction (just
image what can of torture you are generating on the poor stack when
creating memory with a destruction phase to throw something).

On 02/22/2016 04:44 PM, Antonio El Khoury wrote:

Our whole code base is supposed to be exception-free. I know this is
not currently the case in pinocchio, which is why this is not a hard
requirement.


Reply to this email directly or view it on GitHub
#108 (comment).

@jcarpent
Copy link
Contributor Author

A robot wrapper have been implemented in https://github.com/stack-of-tasks/tsid

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