Gazebo Tutorial and partial Oops assignment added. #2
Conversation
|
@shivamvats Is this the complete commit? I don't see some auxiliary files (CMakeLists.txt and manifest.xml). Are they not required? Also, if this is a package, it would be nice to see some structure (source files under 'src', includes under 'include', configuration files under 'config' etc). Other than that, nice work! 👍 |
… working on debugging
…ependency related errors.
|
@shivamvats why are there two folders |
|
@shivamvats there is no |
| #include <opencv/highgui.h> | ||
| #include <opencv2/imgproc/imgproc.hpp> | ||
| #include "queue.h" | ||
|
|
There was a problem hiding this comment.
You have not included Oops_class.h in this file.
|
How do I delete commited files? Should I add the folder after deleting it or should I rm the folder? |
|
Delete the files and commit (you have to commit that you have deleted) On Sat, Mar 15, 2014 at 7:43 AM, shivamvats [email protected]:
|
| #ifndef queue_h | ||
| #define queue_h | ||
| #include <iostream> | ||
| #include "queue.cpp" |
There was a problem hiding this comment.
Remove this line.
The reason you are getting error like
queue.cpp:11:6: error: ‘queue’ has not been declared are because you include queue.h in queue.cpp and in queue.cpp you have included queue.h. This lead to circular dependency.
There was a problem hiding this comment.
After doing I was able to at least compile your code.
Good Job 👍
|
Add the image you used for testing. And avoid hardcoding the locations of files in the code. |
No description provided.