PlotJuggler icon indicating copy to clipboard operation
PlotJuggler copied to clipboard

Latest code crashes with old layout files

Open ssys-zdavis opened this issue 1 year ago • 1 comments

Problem description

A recent change: https://github.com/facontidavide/PlotJuggler/commit/36cebbf680db4310720fd71846657b23ed52bbd9#r138442242

broke loading of old layouts that have a "Scale/Offset" transform applied.

Describe the problem and the expected behavior. Use screenshots or a video, if applicable.

Answer these questions (or I will ignore your issue)

  1. Describe your platform / Operative System and the version of PlotJuggler you are using.

  2. Compile and try the latest version in the main branch and check if the problem persists.

  3. Check if the problem can be reproduced using the dummy data created by the command line argument "-t" or one of the files in the folder "datasamples".

  4. If it can't be reproduced with the dummy data, please share the CSV file or the rosbag that can be used to reproduce the problem.

  5. Linux and Windows with latest main

  6. It does (see 1)

  7. Yes. Run with -t and open the attached layout file

broken_layout.xml.txt

ssys-zdavis avatar Feb 08 '24 20:02 ssys-zdavis

Thanks for reporting... I did not think about that problem of back compatibility.

I will fix it soon

facontidavide avatar Feb 08 '24 21:02 facontidavide