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.

Sign up for GitHub

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

Avoid double timestamps in debug log output #129

Merged
merged 3 commits into from
Aug 17, 2023
Merged

Avoid double timestamps in debug log output #129

merged 3 commits into from
Aug 17, 2023

Commits on Aug 17, 2023

  1. Don't include name of the day in debug log stamp 

    ie: 'MON', 'TUE', etc.

Makes the length of the stamp more predictable.

And append 6 digits of the nanosecond part of the stamp instead of 3 (we have the resolution).
    @gavanderhoorn
    gavanderhoorn committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    02437c7 View commit details
    Browse the repository at this point in the history

  2. Avoid printing both sent and receive stamp in the logger 

    PR Yaskawa-Global#61 added a timestamp to the log message sent out by MotoROS2. Use that timestamp instead of calculating a new one upon reception.

The format of the timestamp prefixed by MotoROS2 is exactly the same as what `debug_listener.py` used to prefix. But due to limitations on the VxWorks side, will likely not take time-zones into account.
    @gavanderhoorn
    gavanderhoorn committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    79fb22e View commit details
    Browse the repository at this point in the history

  3. They're 'sinks', not 'syncs' 

    So correct method name.
    @gavanderhoorn
    gavanderhoorn committed Aug 17, 2023
    Configuration menu
    Copy the full SHA
    277aa1c View commit details
    Browse the repository at this point in the history