Skip to content

Conversation

@EkaterinaMarova
Copy link

Added benchmark folder with a benchmark for ros2 reduct agent vs. rosbag2.

@EkaterinaMarova EkaterinaMarova linked an issue Jul 11, 2025 that may be closed by this pull request
Copy link
Member

@AnthonyCvn AnthonyCvn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the initial benchmarking setup; looks solid !

A few suggestions:

  • 🔄 Instead of using a .ipynb, please create a benchmark.py script and make it interactive using # %% cells. This will keep it notebook-friendly while being easier to version and execute in VS Code or other editors.
  • 📦 Use the following link to download the test dataset (public link on our google drive):
    https://drive.google.com/file/d/1gaQSR3mhrhTSSjqdcu8M-kdCHiwh4d4J
    And please add *.mcap to the .gitignore so the file isn't committed. + you can add the YAML file in the repo (and update the paths like "/home/ekaterina/Downloads/" to relative import like "./benchmark/...")
  • ✅ The current benchmarking logic is good; however, we only need to compare with --compression-mode=message in rosbag2. File compression is not supported with the agent.
  • 📊 For the ros2_reduct_agent benchmarks, also include system resource monitoring for ReductStore. This way, we're comparing rosbag2 record vs agent + reductstore.

@AnthonyCvn AnthonyCvn mentioned this pull request Jul 28, 2025
1 task
@AnthonyCvn AnthonyCvn assigned AnthonyCvn and unassigned AnthonyCvn Jul 31, 2025
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.

Benchmark ros2_reduct_agent and rosbag2

3 participants