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

Publish rectified images #14

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Publish rectified images #14

wants to merge 6 commits into from

Conversation

ebimor
Copy link

@ebimor ebimor commented May 10, 2018

Hello, I have added some changes to your code. It can now read the SNxxx.conf file and based on that create undistortion mappings. These mappings are then used to rectify the image and publish them.

Copy link
Owner

@willdzeng willdzeng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few comments.
Thanks for your contribution

@@ -17,496 +17,483 @@
#define HEIGHT_ID 4
#define FPS_ID 5

namespace arti
{
namespace arti {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you run clang-format on this file with the config .clang-format in the repository so that I can see the diff?
Thanks.

@@ -12,14 +12,14 @@ find_package(catkin REQUIRED
)
# set(CMAKE_CXX_FLAGS "-std=c++0x ${CMAKE_CXX_FLAGS}")

# find_package(Eigen3 REQUIRED)
find_package(Eigen3 REQUIRED)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you might need to add eigen in .travis.yml file for the build to pass

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.

3 participants