Skip to content

Add analyze map connected components and suggested probes#790

Merged
yuechen-li-dev merged 1 commit into
masterfrom
codex/add-connected-components-and-probes-to-analyze-map
Jun 23, 2026
Merged

Add analyze map connected components and suggested probes#790
yuechen-li-dev merged 1 commit into
masterfrom
codex/add-connected-components-and-probes-to-analyze-map

Conversation

@yuechen-li-dev

Copy link
Copy Markdown
Owner

Motivation

  • Make aetheris analyze map --views six --llm --json produce machine-readable, conservative topology hints so LLMs do not have to manually parse ASCII compact grids to find holes/plateaus/curved regions.
  • Provide actionable next steps by suggesting ready-to-copy follow-up point probes (CLI commands) that disambiguate interior no-hit islands, curved clusters, and approximate/tessellated regions.

Description

  • Added new Six-view schema records (SixViewMapComponents, MapComponent, CellBoundingBox, SuggestedMapProbe) to StepAnalysisModels.cs and extended SixViewMapResult / SixViewMapView to carry per-view components and per-view/top-level suggested probes.
  • Implemented conservative 4-connected component analysis in StepAnalyzer.cs (FindComponents + BuildSixViewComponents) that groups: no-hit components, height/depth band components (rounded scalar bands), surface-family components, and tessellated-fallback components, computing bbox/cell counts/centroids/backend dominance and confidence hints.
  • Implemented BuildSuggestedProbes to emit centroid-based probe suggestions (per-view and a bounded global list) with ready-to-copy CLI commands; applied output limits and truncation reporting (max 10 components/category/view, max 10 probes/view, max 30 global probes with truncated/omittedCount).
  • Updated measured summary wording to conservatively mention interior no-hit candidates, border-touching silhouette gaps, and curved-family clusters; added A5 docs and forward links; updated tests to validate new fields; explicitly does not perform feature reconstruction or assert semantic features as facts.

Testing

  • Ran dotnet restore Aetheris.slnx and dotnet build Aetheris.slnx -f net10.0 --no-restore /m:1 and the build succeeded without errors.
  • Ran focused CLI unit tests dotnet test Aetheris.CLI.Tests/Aetheris.CLI.Tests.csproj -f net10.0 --filter "AnalyzeMap|SixView|Components|Probes|Torus|Cylinder|Fallback|Ruled" which passed (13 tests).
  • Performed CLI smoke runs: aetheris analyze map testdata/step242/nist/FTC/nist_ftc_09_asme1_ap242-e1.stp --views six --resolution 16x16 --llm --json produced a six-view-summary with 6 views, 26 global suggested probes and 59 total reported components; aetheris analyze map testdata/step242/generated/ruled-a2/ellipse-linear-extrusion-production.step --views six --resolution 8x8 --llm --json produced a six-view summary with components and probes (6 global probes, 6 total components).
  • Also validated dotnet run --project Aetheris.CLI -- --help; git diff --check and repository status were clean after commit.

Codex Task

@yuechen-li-dev yuechen-li-dev merged commit 9f692bb into master Jun 23, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant