-
Notifications
You must be signed in to change notification settings - Fork 52
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
Add support / test with Eigen 3.3 #201
Labels
Comments
Should we change from previous version to Eigen 3.3 the code or have like a automatic switch (even with a user-defined command) and keep the older and 3.3 versions together? I would vote second option, because for example in Ubuntu 14.04 Eigen3 is the default one, but in Ubuntu 16.04 Eigen3.3.. |
I vote for the second option.
On Mon, Mar 6, 2017 at 10:40 AM Konstantinos Chatzilygeroudis < ***@***.***> wrote:
Should we change from previous version to Eigen 3.3 the code or have like
a automatic switch (even with a user-defined command) and keep the older
and 3.3 versions together? I would vote second option, because for example
in Ubuntu 14.04 Eigen3 is the default one, but in Ubuntu 16.04 Eigen3.3..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#201 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AFu8v2Ok8S2CQtoEykc-yLmtrJ_x0S8eks5ri9N_gaJpZM4LYVN6>
.
--
-- Jean-Baptiste Mouret
ERC ResiBots: http://www.resibots.eu
Web: http://members.loria.fr/JBMouret
|
Merged
Done in #225 ... |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
see: http://eigen.tuxfamily.org/index.php?title=3.3
(this could make our code faster & more numerically stable).
The text was updated successfully, but these errors were encountered: