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

Rostime algorithm errors #15

Closed
stonier opened this issue Mar 15, 2013 · 6 comments
Closed

Rostime algorithm errors #15

stonier opened this issue Mar 15, 2013 · 6 comments
Assignees
Milestone

Comments

@stonier
Copy link
Member

stonier commented Mar 15, 2013

ros/ros_comm#160

@ghost ghost assigned stonier Mar 15, 2013
@stonier
Copy link
Member Author

stonier commented Mar 15, 2013

  • Download his zipped standalone test code
  • Compare with current winros code in rostime
  • If it's the same code, then
    • Compile and execute
    • If reproducible, debug.

@yichoe
Copy link

yichoe commented Jun 18, 2013

Time::now() returns g_sim_time as well as throws TimeNotInitializedException unless Time::init() is called. Since 'straszheim' didn't call Time::init(), ros_walltime() wouldn't be executed.

When we call Time::now() after calling Time::init(), the second part is updated well.

I wonder how straszheim got the result using MSVC.

@stonier
Copy link
Member Author

stonier commented Jun 27, 2013

time::init() gets called in ros::init(), so it should be called.

Conclusion - we can't reproduce.

@stonier
Copy link
Member Author

stonier commented Jun 27, 2013

Made a recommendation to close the issue on ros_comm.

@stonier stonier closed this as completed Jun 27, 2013
@yichoe
Copy link

yichoe commented Jun 28, 2013

PS: Although ros::init() is called, TimeNotInitializedException is raised. In ros::init(), Time::init() does not appear as well.

@stonier
Copy link
Member Author

stonier commented Jun 28, 2013

Just checked - you are right. Instead it gets called the first time a Nodehandle is constructed (libros/init.cpp -> libros/NodeHandle.cpp).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants