Skip to content
This repository was archived by the owner on May 31, 2025. It is now read-only.

Add timestamp formatting for rosconsole#1458

Closed
abrzozowski wants to merge 1 commit intoros:kinetic-develfrom
abrzozowski:kinetic-devel
Closed

Add timestamp formatting for rosconsole#1458
abrzozowski wants to merge 1 commit intoros:kinetic-develfrom
abrzozowski:kinetic-devel

Conversation

@abrzozowski
Copy link
Copy Markdown
Contributor

@abrzozowski abrzozowski commented Jul 14, 2018

Add timestamp formater, specified by ROSCONSOLE_FORMAT environment variable.

You can change timestamp format, by typing data format codes after colon of time or walltime in ROSCONSOLE_FORMAT.

For example:

<:$ export ROSCONSOLE_FORMAT='${time:%Y-%m-%d %H:%M:%S}: ${message}' && rosrun rosconsole_test ab 
2018-07-14 13:35:49: hello world 0
2018-07-14 13:35:50: hello world 1
2018-07-14 13:35:51: hello world 2

shall ensure backward compatibility:

<:$ export ROSCONSOLE_FORMAT='${time}: ${message}' && rosrun rosconsole_test ab 
1531575287.875628208: hello world 0
1531575288.875585211: hello world 1
1531575289.878066158: hello world 2

@abrzozowski abrzozowski force-pushed the kinetic-devel branch 7 times, most recently from 7871a3c to ff94d85 Compare July 15, 2018 16:31
@dirk-thomas
Copy link
Copy Markdown
Member

dirk-thomas commented Nov 10, 2018

Please target the latest development branch (currently melodic-devel) as well as include unit test coverage for the new feature.

@abrzozowski abrzozowski mentioned this pull request Nov 18, 2018
Merged
@abrzozowski
Copy link
Copy Markdown
Contributor Author

abrzozowski commented Nov 18, 2018 •
edited
Loading

@dirk-thomas so PR moved to the rosconsole repository ros/rosconsole#22 for cpp formater and to #1533 for python part.

@abrzozowski abrzozowski mentioned this pull request Nov 18, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement requires-changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@abrzozowski @dirk-thomas