Skip to content

feat(metrics): add BDR, RaR, VR, SVF and AR urban form indices - #4

Merged
kastnerp merged 1 commit into
devfrom
claude/implementation-questions-e32354
Sep 21, 2026
Merged

kastnerp merged 1 commit into
devfrom
claude/implementation-questions-e32354

Conversation

@kastnerp

Copy link
Copy Markdown
Member

What

Adds the remaining urban form indices from the reference table to the morphology metrics, exposed in the Morph text output and the case manifest JSON.

  • BDR: built volume / site area
  • RaR: (site − all parcels) / site area
  • VR: facade area / site area
  • SVF: mean ground-level sky view factor, ray-cast (~400 open-ground points × 144 cosine-weighted rays) against masses and optional terrain
  • AR: mean building height / StreetWidth
  • Existing plan area density and weighted mean height relabelled BCR and BHt (manifest keys unchanged).

Reviewer notes

  • Manifest schema bumped 1.1.0 → 1.2.0; schema version feeds the case ID, so IDs differ from 0.1.0.5.
  • No ports added; only the Morph output description text changed.
  • Build and 60 core tests pass, but the SVF ray-cast, facade and road-area paths were not run in Rhino — please check SVF solve time on a large case. Tune SkyViewSampleTarget/SkyViewRayCount if slow.
  • SVF limits: trees do not occlude; no context outside the boundary, so edge points read high.
  • Template integrity tests not run.

🤖 Generated with Claude Code

Extend morphology metrics with built density ratio, road area ratio,
vertical area ratio, ray-cast ground sky view factor and aspect ratio.
Label existing plan area density and weighted height as BCR and BHt.
Bump case manifest schema to 1.2.0.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@kastnerp
kastnerp requested a review from karadagi September 20, 2026 15:53
@kastnerp
kastnerp merged commit dbb074d into dev Sep 21, 2026
4 checks passed
@kastnerp
kastnerp deleted the claude/implementation-questions-e32354 branch September 21, 2026 03:41
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.

1 participant