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

Not fetching timestamp for x axis? #202

Closed
msadowski opened this issue Sep 20, 2019 · 5 comments
Closed

Not fetching timestamp for x axis? #202

msadowski opened this issue Sep 20, 2019 · 5 comments
Assignees

Comments

@msadowski
Copy link

msadowski commented Sep 20, 2019

Hi Davide!

Thanks for making PlotJuggler, I'm a big fan! I'm using the released version of PlotJuggler so apologies if it has been fixed already.

Recently when playing back some data from bag files I've noticed what seems like time series not being loaded from message timestamp, resulting in plots like the one on the top in this screenshot:

image

Basically when plotting the /rr_robot/mobile_base_controller/odom* fields time starts at 0, while when I plot /rs_t265/imu/* fields the time used is the one we can find in timestamp.

Looking at the value of timestamps in both messages they seem to match each other relatively well:
image

Here is the bag that I used.
bag_filtered.zip

Thanks in advance for any help!

@facontidavide facontidavide self-assigned this Sep 20, 2019
@facontidavide
Copy link
Owner

Interesting. I will have a look

@facontidavide
Copy link
Owner

I am not able to reproduce the error using the latest version on master not binary version 2.3.0 (kinetics)

image

@facontidavide facontidavide removed the bug label Sep 20, 2019
@msadowski
Copy link
Author

Hi Davide!

This is interesting. I'm on Ubuntu 18.04 and running ROS Melodic. I just built the latest master and I was still able to reproduce it. I've recorded my screen while reproducing this in case it's useful for debugging: https://youtu.be/mFuenFWB0k8.

I'm happy to place some debug statements in the code to get to the source of the issue, if you have any hints where to start then let me know!

@facontidavide
Copy link
Owner

That was a tough bug, that was visible only in streaming. Thanks for reporting

@msadowski
Copy link
Author

That was quick! Thanks!!

I just tested it and it works like a charm now!

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

No branches or pull requests

2 participants