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

Dataset for testing #6

Open
manhha1402 opened this issue Aug 15, 2019 · 4 comments
Open

Dataset for testing #6

manhha1402 opened this issue Aug 15, 2019 · 4 comments
Labels
enhancement New feature or request

Comments

@manhha1402
Copy link
Contributor

manhha1402 commented Aug 15, 2019

Hi,
Thank you for nice work.
Can you provide the dataset for testing the package or an example (possibly how to test by using a set of RGB-D images)?

@schornakj
Copy link
Contributor

An example that works out-of-the-box is definitely an important feature, and it's a work in progress. We're currently working towards using the yak_ros wrapper node in combination with gl_depth_sim to provide simulated 3D data. If you want to take a look at the current (slightly rough) state of things you can check out my yak_ros feature/add-ros1-demo-node branch.

This approach would also work with a rosbag for recorded images and transform data. A ROS-independent example would be a little further down the road, since our current work has us using this library exclusively within ROS environments.

@Sinchiguano
Copy link

Hi,

I am so sorry for this simple question, I tried to compile the yak_ros, but i have problem. It threw me the following message:

--- stderr: yak_ros
:0:1: error: macro names must be identifiers
/home/casch/ros2_ws/src/yak_ros/src/yak_ros2_node.cpp: In constructor ‘Fusion::Fusion(rclcpp::Node::SharedPtr, const kfusion::KinFuParams&, const Affine3f&, std::cxx11::string)’:
/home/casch/ros2_ws/src/yak_ros/src/yak_ros2_node.cpp:120:30: warning: ‘Fusion::params
’ will be initialized after [-Wreorder]
const kfusion::KinFuParams params
;

Could anyone of you please give me an idea of what it is wrong, I got ROS 2 Dashing distribution installed. The Yak packages compile successfully with some warning but i think it should be fine.

Best regards
Cesar

@manhha1402
Copy link
Contributor Author

Hi @schornakj
Thank you for considering the example. I think bag file is also a good option. Since I am also working on KinectFusion approach, so it's would be helpful to test the yak package.

@schornakj schornakj added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Nov 6, 2019
@schornakj
Copy link
Contributor

@ipa-mah The yak_ros package now contains a simple demo node that streams simulated depth images of the classic "bunny" mesh. It's not really standalone in the context of the yak library itself, but I think it's a good starting point.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants