Skip to content

Add support for new sensors#72

Merged
GreatAlexander merged 11 commits intomainfrom
new-sensors-configuration
Aug 25, 2025
Merged

Add support for new sensors#72
GreatAlexander merged 11 commits intomainfrom
new-sensors-configuration

Conversation

@hect95
Copy link
Collaborator

@hect95 hect95 commented Aug 7, 2025

  • Add support for new lidars and extra two cameras
    • Install tartan_rosbag_exporter version 2.0.0. to support
      new Hesai lidars pointcloud export.
      • Update av_sensor_export_config.yaml to add timestamp_offset_ms
        parameter for each camera.
    • Update sample_formats.py to include fsp_r and rsp_r cameras.
      • Defining a new 2x4 grid visualisation layout for the new cameras
        in Segments.ai.
    • Update sensor_frame_creator.py to use lidar_top as
      self.LIDAR_FRAME_ID instead of lidar_ouster_top.
  • Update dataset_attributes.json file
    • Sort labels in alphabetical order
  • Skip sequence if poses and keyframes count differ
    • Allow continuing the dataset creation if any sequence's
      trajectory pose count and lidar keyframes differ.
    • Define a new error type InvalidPoseCountError in response.py to
      distinguish this case from Segments.ai errors.
    • Update SegmentsSampleCreator.add() method in
      add_segmentsai_sample.py to prevent continuing with the sample
      creation if this error is raised, and return an appropriate response.
    • Update DatasetCreator.create() in dataset_creator.py to track
      how and which sequences were skipped and log it after processing
      all sequences
  • Update README
    • Fix typo in env variable names
    • Display an image showing the Data pipeline system diagram
    • Re-structure the file so that the Usage guide on how to run the pipeline
      appears first

hect95 added 4 commits August 7, 2025 16:22
- Allow continuing the dataset creation if any sequence's
  trajectory pose count and lidar keyframes differ.
- Define a new error type `InvalidPoseCountError` in `response.py` to
  distinguish this case from segments errors.
- Update `SegmentsSampleCreator.add()` method in
  `add_segmentsai_sample.py` to prevent continuing with the sample
  creation if this error is raised and return an appropriate response.
- Update `DatasetCreator.create()` in `dataset_creator.py` to track
  how and which sequences were skipped and log it after processing
  all sequences.
- Install `tartan_rosbag_exporter` version 2.0.0. to support
  new hesai lidars pointcloud export.
  - Update `av_sensor_export_config.yaml` to add `timestamp_offset_ms`
    parameter for each camera.
- Update `sample_formats.py` to include `fsp_r` and `rsp_r` cameras.
  - Defining a new 2x4 grid visualisation layout for the new cameras
    in Segments.ai.
- Update `sensor_frame_creator.py` to use `lidar_top` as
  `self.LIDAR_FRAME_ID` instead of `lidar_ouster_top`.
- Sort labels in alphabetical order
@hect95 hect95 self-assigned this Aug 7, 2025
@hect95 hect95 marked this pull request as draft August 7, 2025 18:41
@hect95 hect95 changed the title New sensors configuration Add support for new lidars and extra two cameras Aug 7, 2025
@mergify mergify bot requested a review from GreatAlexander August 7, 2025 18:41
@hect95 hect95 changed the title Add support for new lidars and extra two cameras Add support fornew sensors Aug 7, 2025
@hect95 hect95 changed the title Add support fornew sensors Add support for new sensors Aug 7, 2025
hect95 and others added 5 commits August 11, 2025 00:59
- Define default parameters values for both
  `dataset_manager.py` and `dataset_creator.py`
  to simplify their use. Assume the scripts
  are lunched inside the docker container.
- Tune parameters to reduce MOLA Odometry skiping top lidar
  pointclouds
- Make `/mnt/sata_ssd_raid/edinburgh` the default directory
  to search for rosbags to upload
- Make `/mnt/vdb/data` the default EIDF's directory to save upoaded
  rosbags
- Add `--logs_directory` argument in `data_manager` and define
 `/opt/ros_ws/logs_tartan_forklfit` as default
- Ignore mcap files located inside compression temporary directory
  - Allow to resume uploads without removing the
    temporary directory manually
@hect95 hect95 linked an issue Aug 13, 2025 that may be closed by this pull request
- Display an image showing the Data pipeline system diagram
- Re-structure the file so the Usage guide on how to run the pipeline
  appears first
@hect95 hect95 marked this pull request as ready for review August 13, 2025 15:52
@GreatAlexander GreatAlexander merged commit 60671af into main Aug 25, 2025
3 checks passed
@GreatAlexander GreatAlexander deleted the new-sensors-configuration branch August 25, 2025 10:36
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

Successfully merging this pull request may close these issues.

Automate data handling when new rosbags have been uploaded to the EIDF

2 participants