lardemua / atom

Calibration tools for multi-sensor, multi-modal robotic systems
GNU General Public License v3.0
253 stars 26 forks source link

#861 New version of merge atom datasets #864

Closed manuelgitgomes closed 8 months ago

manuelgitgomes commented 8 months ago

Hello @miguelriemoliveira.

I have written a new version of a script to merge ATOM datasets. This now accepts various datasets simultaneously. This addresses #861

miguelriemoliveira commented 8 months ago

Hi @manuelgitgomes ,

I am lost here. How did you copy the data files?

I solved it like this:

https://github.com/lardemua/atom/blob/c8a2e8a29f8daef60c1d8a55723188362bda96f9/atom_calibration/scripts/utilities/merge_atom_datasets#L75-L109

but it looks that you only fuse the json file. We also need to copy the files and change the data_file fields ...

miguelriemoliveira commented 8 months ago

We decided to keep the current version as it is working fine.