Skip to content

Fix compilation error with Eigen 3.3+#1538

Merged
jdumas merged 1 commit intolibigl:masterfrom
bbrrck:master
Jun 5, 2020
Merged

Fix compilation error with Eigen 3.3+#1538
jdumas merged 1 commit intolibigl:masterfrom
bbrrck:master

Conversation

@bbrrck
Copy link
Copy Markdown
Contributor

@bbrrck bbrrck commented Jun 4, 2020

Fixes the compilation error "incompatible operand types" when libigl is compiled with Eigen 3.3+.

Check all that apply (change to [x])

  • All changes meet libigl style-guidelines.
  • Adds new .cpp file.
  • Adds corresponding unit test.
  • This is a minor change.

Fixes the error "incompatible operand types" when libigl is compiled with Eigen 3.3+.
@jdumas
Copy link
Copy Markdown
Collaborator

jdumas commented Jun 4, 2020

Thanks! Is the error happening with 3.3.7? Libigl is already being compiled with this version so I'm surprised this isn't caught by our CI.

@bbrrck
Copy link
Copy Markdown
Contributor Author

bbrrck commented Jun 5, 2020

Thanks! Is the error happening with 3.3.7? Libigl is already being compiled with this version so I'm surprised this isn't caught by our CI.

Good point. The error does not happen in 3.3.7, I was using a more recent snapshot from December 2019 (specifically this tree).

@jdumas
Copy link
Copy Markdown
Collaborator

jdumas commented Jun 5, 2020

Any reason you're not using the stable version (just curious)?

@jdumas jdumas merged commit 48a7e8c into libigl:master Jun 5, 2020
@bbrrck
Copy link
Copy Markdown
Contributor Author

bbrrck commented Jun 5, 2020

Yep, Eigen::all 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants