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

catkin_make failed after jetson-inference update #14

Closed
aparwal opened this issue Jul 23, 2019 · 7 comments
Closed

catkin_make failed after jetson-inference update #14

aparwal opened this issue Jul 23, 2019 · 7 comments

Comments

@aparwal
Copy link

aparwal commented Jul 23, 2019

Could not build ros package after installing latest version of jetson-inference

Output of catkin_make:

/home/nvidia/catkin_ws/src/ros_deep_learning/src/node_detectnet.cpp: In function ‘void img_callback(const ImageConstPtr&)’:
/home/nvidia/catkin_ws/src/ros_deep_learning/src/node_detectnet.cpp:74:119: error: no matching function for call to ‘detectNet::Detect(float*, uint32_t, uint32_t, float*&, int*, float*&)’
  const bool result = net->Detect(cvt->ImageGPU(), cvt->GetWidth(), cvt->GetHeight(), bbCPU, &numBoundingBoxes, confCPU);
                                                                                                                       ^
In file included from /home/nvidia/catkin_ws/src/ros_deep_learning/src/node_detectnet.cpp:29:0:
/usr/local/include/jetson-inference/detectNet.h:271:6: note: candidate: int detectNet::Detect(float*, uint32_t, uint32_t, detectNet::Detection**, uint32_t)
  int Detect( float* input, uint32_t width, uint32_t height, Detection** detections, uint32_t overlay=OVERLAY_BOX );
      ^
/usr/local/include/jetson-inference/detectNet.h:271:6: note:   candidate expects 5 arguments, 6 provided
/usr/local/include/jetson-inference/detectNet.h:283:6: note: candidate: int detectNet::Detect(float*, uint32_t, uint32_t, detectNet::Detection*, uint32_t)
  int Detect( float* input, uint32_t width, uint32_t height, Detection* detections, uint32_t overlay=OVERLAY_BOX );
      ^
/usr/local/include/jetson-inference/detectNet.h:283:6: note:   candidate expects 5 arguments, 6 provided
/home/nvidia/catkin_ws/src/ros_deep_learning/src/node_detectnet.cpp:77:7: error: in argument to unary !
  if( !result )
       ^
/home/nvidia/catkin_ws/src/ros_deep_learning/src/node_detectnet.cpp: In function ‘int main(int, char**)’:
/home/nvidia/catkin_ws/src/ros_deep_learning/src/node_detectnet.cpp:205:20: error: class detectNet’ has no member named ‘GetMaxBoundingBoxes’
  maxBoxes   = net->GetMaxBoundingBoxes();  
                    ^
ros_deep_learning/CMakeFiles/detectnet.dir/build.make:62: recipe for target 'ros_deep_learning/CMakeFiles/detectnet.dir/src/node_detectnet.cpp.o' failed
make[2]: *** [ros_deep_learning/CMakeFiles/detectnet.dir/src/node_detectnet.cpp.o] Error 1
CMakeFiles/Makefile2:7449: recipe for target 'ros_deep_learning/CMakeFiles/detectnet.dir/all' failed
make[1]: *** [ros_deep_learning/CMakeFiles/detectnet.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2
Invoking "make -j6 -l6" failed

Seems like signature of detectNet::Detect() changed after this commit of jetson-inference but this package is using the old signature.
Is there a tag/commit of jetson-inference for which this package works?

@dusty-nv
Copy link
Owner

Hi @aparwal, sorry - looks like I need to update this ROS node.

For now you should be able to use the L4T-R32.1 tag of jetson-inference, that was captured before these changes were merged into master.

@aparwal
Copy link
Author

aparwal commented Jul 24, 2019

Thanks, the node works for that tag.

@aparwal aparwal closed this as completed Jul 24, 2019
@aparwal aparwal reopened this Jul 24, 2019
@agoila
Copy link

agoila commented Jul 27, 2019

I am sorry, but I am relatively new to GitHub, and have been having difficulty understanding what you mean by tag? Is this the same as branch?

I ran into the same issue last night, and am wondering if I need to checkout the branch L4T-R32.1, and follow the build steps again (for jetson-inference) to get this node to work?

Thanks for your help @dusty-nv @aparwal !

@luo980
Copy link

luo980 commented Jul 31, 2019

Found the same problem, but I will wait for the update :D
Thanks

@sgiraz
Copy link

sgiraz commented Aug 1, 2019

Looking at Detect method in "detectNet.h" and the same Detect method used in "node_detectnet.cpp", jetson-inference repo doesn't seems aligned with the ros_deep_learninig repo.
I'm waiting for the update too!! ;)

@neurondizzy
Copy link

Hello kmartin .... just used your update and all went well with build, waiting on a camera to move further. Thanks for the update.

@dusty-nv
Copy link
Owner

The ros_deep_learning repo has now been updated for JetPack 4.2 and to build against jetson-inference master. Let me know if you have any more problems building it.

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

No branches or pull requests

6 participants