Falling-Metal-Interactive/Blacksite
2
0
Fork 0
Code Issues 15 Pull Requests Actions Packages Projects Releases Wiki Activity

BS-JD-303 - Terrain material layers and texture painting MVP #24

New Issue
Closed
opened 2026-06-06 01:48:06 -04:00 by Rbanh · 1 comment
Rbanh commented 2026-06-06 01:48:06 -04:00
Owner
Copy Link

Summary

Add terrain material layers with compact normalized weight painting.

Delivered

  • Backward-compatible TerrainDesc support for up to four shared Material/Material Instance layers.
  • Deterministic RGBA8 weights per height sample with exact normalization and implicit layer-zero defaults.
  • Vertex-color weight transport and revision-aware blended PBR hydration through blacksite_surface.
  • Terrain inspector assignment, clear, reorder, remove, UV tiling, drag/browse targets, and sample coverage.
  • Modal Paint/Erase in the existing horizontal viewport toolbar with live footprint, exact cancel, and one history transaction per stroke.
  • Generic reflected-history replacement fix for shrinking list-backed component snapshots.
  • Deterministic two-layer terrain showcase, ADR 0040, workflow docs, tests, and native Wayland evidence.

Acceptance criteria

  • Multiple terrain material layers can be assigned.
  • User can paint layer weights with preview.
  • Weight data persists and hydrates consistently.
  • Invalid/missing material refs report clearly and retain a visible fallback.

Tests/docs

  • Layer serialization and validation tests.
  • Vertex-weight transport and material packing tests.
  • Paint/Erase normalization, footprint isolation, cancel, and single-history undo/redo tests.
  • Full source gate and live native-Wayland acceptance.
  • Terrain guide, root controls/checklist, ADR 0040, working plan, and evaluation record updated.

Integrated on main in d1a56f7. Packaged tests were intentionally skipped per current project direction.

## Summary Add terrain material layers with compact normalized weight painting. ## Delivered - Backward-compatible `TerrainDesc` support for up to four shared Material/Material Instance layers. - Deterministic RGBA8 weights per height sample with exact normalization and implicit layer-zero defaults. - Vertex-color weight transport and revision-aware blended PBR hydration through `blacksite_surface`. - Terrain inspector assignment, clear, reorder, remove, UV tiling, drag/browse targets, and sample coverage. - Modal Paint/Erase in the existing horizontal viewport toolbar with live footprint, exact cancel, and one history transaction per stroke. - Generic reflected-history replacement fix for shrinking list-backed component snapshots. - Deterministic two-layer terrain showcase, ADR 0040, workflow docs, tests, and native Wayland evidence. ## Acceptance criteria - [x] Multiple terrain material layers can be assigned. - [x] User can paint layer weights with preview. - [x] Weight data persists and hydrates consistently. - [x] Invalid/missing material refs report clearly and retain a visible fallback. ## Tests/docs - [x] Layer serialization and validation tests. - [x] Vertex-weight transport and material packing tests. - [x] Paint/Erase normalization, footprint isolation, cancel, and single-history undo/redo tests. - [x] Full source gate and live native-Wayland acceptance. - [x] Terrain guide, root controls/checklist, ADR 0040, working plan, and evaluation record updated. Integrated on `main` in `d1a56f7`. Packaged tests were intentionally skipped per current project direction.
terrain-material-paint-stroke.png
1.1 MiB
terrain-material-paint-stroke.png
Rbanh added this to the M3 - Terrain and physics placement milestone 2026-06-06 01:48:06 -04:00
Rbanh added the
roadmap:jackdaw
type:feature
priority:P2
area:terrain
 labels 2026-06-06 01:48:06 -04:00
Rbanh commented 2026-07-12 21:03:51 -04:00
Author
Owner
Copy Link

Implemented and integrated on main in d1a56f7.

Terrain material Paint acceptance

Live native-Wayland acceptance:

  • Concrete and Surface Tint blend continuously across generated terrain chunks.
  • The Terrain inspector exposes two shared layer assignments, UV tiling, reorder/remove/clear, drag/browse targets, and coverage counts.
  • Paint/Erase stays in the existing horizontal viewport toolbar with target layer, radius, and strength.
  • A drag changed layer-two coverage from 13/25 to 16/25 samples and produced exactly one Undo: Paint Terrain Material; Ctrl+Z restored 13/25 and Ctrl+Y restored 16/25.
  • The terrain-following cyan footprint rendered and the transform gizmo stayed suppressed while Paint owned input.
  • Forward PBR compiled without GPU pipeline errors; QA also removed an invalid deferred-only prepass override from both project material extensions.

Source gate: fmt, workspace check, strict workspace clippy, full workspace tests, cargo validate-levels (60 dependencies, 5 existing non-blocking findings, 0 blockers), cargo bake-navigation (1 artifact), and diff hygiene all pass. Packaged tests remain intentionally skipped per current direction.

Implemented and integrated on `main` in `d1a56f7`. ![Terrain material Paint acceptance](https://git.spacetrainclubhouse.com/attachments/84b7b801-a7ff-42fb-a551-9b2da9d6fcb1) Live native-Wayland acceptance: - `Concrete` and `Surface Tint` blend continuously across generated terrain chunks. - The Terrain inspector exposes two shared layer assignments, UV tiling, reorder/remove/clear, drag/browse targets, and coverage counts. - Paint/Erase stays in the existing horizontal viewport toolbar with target layer, radius, and strength. - A drag changed layer-two coverage from 13/25 to 16/25 samples and produced exactly one `Undo: Paint Terrain Material`; Ctrl+Z restored 13/25 and Ctrl+Y restored 16/25. - The terrain-following cyan footprint rendered and the transform gizmo stayed suppressed while Paint owned input. - Forward PBR compiled without GPU pipeline errors; QA also removed an invalid deferred-only prepass override from both project material extensions. Source gate: fmt, workspace check, strict workspace clippy, full workspace tests, `cargo validate-levels` (60 dependencies, 5 existing non-blocking findings, 0 blockers), `cargo bake-navigation` (1 artifact), and diff hygiene all pass. Packaged tests remain intentionally skipped per current direction.
Rbanh closed this issue 2026-07-12 21:04:13 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
area:animation

Skeletal animation import, preview, graph/state authoring, and runtime integration.

  
area:assets

Asset browser, material browser, thumbnails, import, processing, and asset watcher.

  
area:audio

Audio source/listener authoring, buses, spatial preview, and runtime integration.

  
area:brushes

Brush geometry, CSG, face editing, UVs, and brush diagnostics.

  
area:build-release

Build profiles, packaging, content validation, release gates, and distribution diagnostics.

  
area:extensions

Static extension API, extension scaffolds, dylib exploration.

  
area:governance

Policy, attribution, planning, and production process.

  
area:hierarchy

Outliner hierarchy, parenting, grouping, prefab structure, ordering, and selection workflows.

  
area:inspector

Inspector, component metadata, component editing UX.

  
area:navigation

Navigation mesh authoring, baking, diagnostics, and AI path preview.

  
area:operators

Editor operator/action framework, undo, commit/cancel semantics.

  
area:physics

Physics placement, colliders, rigid bodies, diagnostics, and overlays.

  
area:project-ux

Project launcher, profiles, templates, or first-run project workflow.

  
area:quality

Regression packs, tests, performance budgets, profiler, first-hour UX.

  
area:rendering

Static and skinned renderers, material slots, shaders, ray tracing, and render-path integration.

  
area:runtime

BRP, remote inspector, PIE, game preview, runtime integration.

  
area:scene-io

Scene document, save/load, schema, migration, or reference repair.

  
area:terrain

Terrain authoring, terrain sculpting, heightmaps, and terrain materials.

  
area:viewport

Viewport interaction, selection, shortcuts, camera, overlays, and controls.

  
priority:P0

Must-have prerequisite or milestone-critical item.

  
priority:P1

Important production feature after P0 work is stable.

  
priority:P2

Useful follow-up or polish item.

  
priority:P3

Exploratory or later spike.

  
roadmap:jackdaw

Jackdaw-inspired Blacksite editor roadmap item.

  
roadmap:production

Blacksite production-readiness roadmap item beyond the Jackdaw-inspired foundation.

  
type:architecture

Architecture, internal framework, or hard-to-reverse technical decision.

  
type:docs

Documentation, policy, samples, or issue-board structure.

  
type:epic

Tracking epic that groups related milestone work.

  
type:feature

User-visible feature or capability.

No Label
area:animation
area:assets
area:audio
area:brushes
area:build-release
area:extensions
area:governance
area:hierarchy
area:inspector
area:navigation
area:operators
area:physics
area:project-ux
area:quality
area:rendering
area:runtime
area:scene-io
area:terrain
area:viewport
priority:P0
priority:P1
priority:P2
priority:P3
roadmap:jackdaw
roadmap:production
type:architecture
type:docs
type:epic
type:feature
Milestone
Clear milestone
No items
No Milestone
M3 - Terrain and physics placement
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Falling-Metal-Interactive/Blacksite#24
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?