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

Check jointDense for all the joints #119

Closed
jcarpent opened this issue Mar 3, 2016 · 6 comments
Closed

Check jointDense for all the joints #119

jcarpent opened this issue Mar 3, 2016 · 6 comments

Comments

@jcarpent
Copy link
Contributor

jcarpent commented Mar 3, 2016

No description provided.

@fvalenza
Copy link
Contributor

Do we still need jointDense with the new JointAccessor ?

@nmansard
Copy link
Contributor

nmansard commented Jun 15, 2016 via email

@jcarpent
Copy link
Contributor Author

An other question is: do we need jointDense converter for all joint types ?
I do not think so as it is just a copy of the sparse quantity to their dense counterpart, while joint accessor does the same job but avoiding copy.

@nmansard
Copy link
Contributor

I agree with that: we do not need conversion any more.
However, would you also agree that now, automatic conversion should be
possible (from JointVariant to JointDense)?

On 06/16/2016 09:43 AM, Justin Carpentier wrote:

An other question is: do we need jointDense converter for all joint
types ?
I do not think so as it is just a copy of the sparse quantity to their
dense counterpart, while joint accessor does the same job but avoiding
copy.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#119 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AAb-CCVIGZmJPd_maIUzEEvFBfgal17Dks5qMP6VgaJpZM4HoiwP.

@jcarpent
Copy link
Contributor Author

With the new JointAccessor, it is now possible to get the conversion of the sparse spatial quantities to their dense representation.

@jcarpent
Copy link
Contributor Author

No need this feature in the very near future. Consider this Issue as solved.

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