PX4 / PX4-Autopilot

PX4 Autopilot Software
https://px4.io
BSD 3-Clause "New" or "Revised" License
8.2k stars 13.37k forks source link

Eigen Include Error #4313

Closed tasarmiento closed 8 years ago

tasarmiento commented 8 years ago

In file included from /Users/TAS/PX4_src/sitl_gazebo/include/gazebo_multirotor_base_plugin.h:29: /Users/TAS/PX4_src/sitl_gazebo/include/common.h:22:10: fatal error: 'Eigen/Dense' file not found

include <Eigen/Dense>

     ^

1 error generated.

In file included from /Users/TAS/PX4_src/sitl_gazebo/src/gazebo_controller_interface.cpp:22: /Users/TAS/PX4_src/sitl_gazebo/include/gazebo_controller_interface.h:23:10: fatal error: 'Eigen/Eigen' file not found

include <Eigen/Eigen>

     ^

1 error generated.

Can be solved by updating the path to: #include <eigen3/Eigen/Dense>. Believe this was solved back in January, but the issue resurfaced today.

LorenzMeier commented 8 years ago

What is the exact diff in your local folder? Sorry about the regression - some well intended cleanup didn't apparently really work.

tasarmiento commented 8 years ago

Lorenz,

The only thing missing was eigen3 before Eigen in the path. I believe this is the same issue that happened back in Jan (posted here: https://groups.google.com/forum/#!searchin/px4users/eigen/px4users/3ttWk0GpwtU/9KTqh7-GFgAJ ), but I just forked that repository today to start working and had the same issue with make.

After fixing the eigen issue I ran into the same optic flow sensor follow-on issue as #4195, but removing Gazebo7 and installing 6 fixed the issue. Somewhere in your previous instructions they noted Mac users must install Gazebo7, but I believe you've since removed that comment.

Thanks for all the great work on this SITL simulation.

-Traci.

On Wed, Apr 20, 2016 at 2:32 PM, Lorenz Meier notifications@github.com wrote:

What is the exact diff in your local folder? Sorry about the regression - some well intended cleanup didn't apparently really work.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/PX4/Firmware/issues/4313#issuecomment-212571143

LorenzMeier commented 8 years ago

Can you please give me the output of git diff in Tools/sitl_gazebo? I don't understand what you changed.

tasarmiento commented 8 years ago

Sorry it was unclear. I did not make changes in Firmware/Tools/sitl_gazebo

Here is the output of git diff from ~/PX4_src/sitl_gazebo where I made the changes:

diff --git a/include/common.h b/include/common.h

index 5d97a44..9ad9984 100644

--- a/include/common.h

+++ b/include/common.h

@@ -19,7 +19,7 @@

*/

-#include <Eigen/Dense>

+#include <eigen3/eigen/Dense>

include <gazebo/gazebo.hh>

namespace gazebo {

_diff --git a/include/gazebo_controller_interface.h b/include/gazebo_controllerinterface.h

index 8ae5c69..f29716b 100644

_--- a/include/gazebo_controllerinterface.h

_+++ b/include/gazebo_controllerinterface.h

@@ -20,7 +20,7 @@

include <boost/bind.hpp>

-#include <Eigen/Eigen>

-Traci.

On Thu, Apr 21, 2016 at 1:53 AM, Lorenz Meier notifications@github.com wrote:

Can you please give me the output of git diff in Tools/sitl_gazebo? I don't understand what you changed.

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub https://github.com/PX4/Firmware/issues/4313#issuecomment-212771393

LorenzMeier commented 8 years ago

Ok, pushed something equivalent to master.