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

Logging meta-ticket #425

Closed
10 of 14 tasks
dhood opened this issue Nov 14, 2017 · 6 comments
Closed
10 of 14 tasks

Logging meta-ticket #425

dhood opened this issue Nov 14, 2017 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@dhood
Copy link
Member

dhood commented Nov 14, 2017

Steps towards the use of logging API and minimal logger severity configuration [limitations in square brackets].

Minimal functionality


Demo/documentation


C++ logger object


Enhancements

  • Configuration of non-node loggers' context
  • Improve ease-of-use of accessing node logger from callbacks
  • Wrapping of "throttled" filter in C++ macros (done in Enable throttling logs rclcpp#879)
  • Unique context of e.g. "once" for each logger object
@dhood dhood added the in progress Actively being worked on (Kanban column) label Nov 14, 2017
@dhood dhood self-assigned this Nov 14, 2017
@dhood dhood added enhancement New feature or request and removed in progress Actively being worked on (Kanban column) labels Feb 26, 2018
@DLu
Copy link

DLu commented Jun 27, 2018

The migration guide really should be updated to include some basic usage. It still says to use printf, and it took me an hour to sort through this to figure out the new syntax, and that was once I found this wiki page.

exhuasted

@wjwwood
Copy link
Member

wjwwood commented Jun 27, 2018

The migration guide really should be updated to include some basic usage.

Sorry the migration guide is out-of-date, but the good news is that it's a wiki so you can edit it:

https://github.com/ros2/ros2/wiki/Migration-Guide/_edit

and it took me an hour to sort through this to figure out the new syntax, and that was once I found this wiki page.

I understand that the migration guide was misleading, but if you just do a search (I did incognito) for "ros2 logging", you get something like this:

screen shot 2018-06-27 at 2 26 25 pm

The first hit is this wiki page that @dhood made that explains how to configure logging and links to examples in both C++ and Python:

https://github.com/ros2/ros2/wiki/Logging

The second hit is a wiki page describing the logging demo she made:

https://github.com/ros2/ros2/wiki/Logging-and-logger-configuration

@wjwwood
Copy link
Member

wjwwood commented Jun 27, 2018

Also, all of our examples and demos use the new logging syntax if you want to look at them for additional examples of using logging, e.g.:

https://github.com/ros2/examples/blob/f6027edfb3a28f012438d1f2d587d0a980e9341c/rclcpp/minimal_subscriber/not_composable.cpp#L27

@DLu
Copy link

DLu commented Jun 27, 2018

Thanks for those links, sincerely. Unfortunately, they did not show up in my Google searches for ros2 ros_info or ros2 rclcpp_info.

david.google_fu--

@wjwwood
Copy link
Member

wjwwood commented Jun 27, 2018

No worries, but we're busy in the middle of a release, so we would also (sincerely) appreciate help updating any documentation.

@clalancette
Copy link
Contributor

Given that we've generally completed logging here, I'm going to close this one out.

There are still things to be done in logging (i.e. ros2/rcl_logging#92), but we should use more targeted issues for those.

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

4 participants