stacks: commit full_default + lite_default wiring baselines for MIGHTY (0.21.0-dev.12) - #427
Merged
Merged
Conversation
… 0.21.0-dev.12 Observed Isaac Sim snapshots (airstack test -m wiring, 1 robot) on the 0.21.0-dev.11 robot image: full_default 86 nodes / 392 edges, lite_default 83 nodes / 378 edges; the MIGHTY module nodes replace droan/disparity_expander_node and the two differ only by the global layer (Container, vdb_mapping, random_walk_node). Both READMEs point at the committed files again. Note: this image ships MAVROS 2.15.1, whose plugin sub-nodes are named interface/mavros/mavros/<plugin> (2.14.0 used interface/mavros/<plugin>); goldens captured on older images (full_droan, full_droan_cpu, full_macvo) will drift on fresh images until regenerated — recorded in the release notes. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
…ommitted Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Contributor
Test Metrics —
|
| Test | Pass | Fail | Skip | Rate |
|---|---|---|---|---|
| test_colcon_build_gcs | 0 | 1 | 0 | 0% |
| test_colcon_build_ms_airsim | 0 | 1 | 0 | 0% |
| test_colcon_build_robot | 0 | 1 | 0 | 0% |
| test_colcon_test_robot | 0 | 1 | 0 | 0% |
Metrics
| Test | Metric | Value |
|---|---|---|
| test_colcon_build_robot | duration_s | 0.79s |
| test_colcon_test_robot | duration_s | 0.644s |
| test_colcon_build_gcs | duration_s | 0.606s |
| test_colcon_build_ms_airsim | duration_s | 0.581s |
Member
Author
|
Note: three commits pushed to this branch after the merge (asm_mighty v0.1.5 re-pin, refreshed baselines, and the CI fix for |
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
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
Commits the two wiring baselines that #424 left to the bootstrap path, from validated Isaac Sim snapshot runs (
airstack test -m wiring --stack <name> --sim isaacsim --num-robots 1) on the 0.21.0-dev.11 robot image built from the currentDockerfile.robot:full_defaultmighty/{global_mapper_ros,mighty_node,mighty_bridge,mighty_init_pose_tf}in,droan/disparity_expander_nodeoutlite_defaultfull_defaultonly byContainer,vdb_mapping,random_walk_nodeThe READMEs point at the committed files again. VERSION → 0.21.0-dev.12 (docs/baselines only; publish retags).
Heads-up: MAVROS 2.15.1 renamed plugin sub-nodes
The fresh image carries MAVROS 2.15.1 (the dev.10 image had 2.14.0). Plugin sub-nodes are now
interface/mavros/mavros/<plugin>instead ofinterface/mavros/<plugin>. Both new goldens were captured on the new image, so they match what CI will observe on a fresh build. The pre-existing goldens (full_droan,full_droan_cpu,full_macvo) were captured on 2.14.0 and will report drift on freshly built images until regenerated (airstack test -m wiring --stack <name>); that is a trunk-wide follow-up, not caused by this PR, and is recorded in the release notes. A first capture offull_defaulton an interim dev.10-based image was discarded for exactly this reason — after normalising the MAVROS names it was node-identical to the committed one.Validation
pytest tests -m wiring --stack full_defaultand--stack lite_default: sentinel check passed, observed snapshots written;full_defaultre-run against its own new golden is the natural next CI check.tests/meta/test_stack_layout_contract.py): 42 passed (H1 names the stack,wiring-graph-v1trailer parses).airstack up --stack <name>, wait for the nodes, then run the wiring mark. A stale colcon tree from a different image fails withNo rule to make target libfastcdr.so.2.2.x— wiperobot/ros_ws/{build,install,log}(root-owned; use a container) first.🤖 Generated with Claude Code