feat(benchmark): add SISO velocity response test#153
Open
xiaolefang-dm wants to merge 11 commits into
Open
Conversation
- benchmark_node: fix double-publish on stop after completion; odom QoS queue depth 20→10; store [t,x,y,z] trajectory; cap scoring distance matrix to 2000 samples; stop publishing target at path end - node_manager: split _benchmark_cmd_vel_proc from nav's _cmd_vel_proc; fix _on_benchmark_result double-merging result into status; add has_odom property; guard cmd_start_benchmark against active map build - routers/benchmark: use has_odom property instead of private field access - benchmark_tab: extract _kBenchmarkAccent constant; fix shouldRepaint to use structural comparison instead of reference equality Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
cmd_stop_benchmark and cmd_start_benchmark no longer touch _global_path, _nav_target_pose, or _nav_paused. Benchmark owns only its own status and result fields; nav state is preserved across a benchmark run so it can be restored when nav resumes. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
# Conflicts: # app/frontend/lib/pages/home_page.dart
|
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Add a SISO velocity-response benchmark for robot control evaluation.
What changed
benchmark_node.pyfor SISO vx sine testing/cmd_veland records odometry responseNotes
This PR intentionally removes the previous 8-shape/PNC benchmark UI path and keeps the benchmark focused on SISO control response.