Skip to content

refactor(assets): clean up noise and heatmap textures - #431

Merged
CloudeaSoft merged 4 commits into
1.4.4from
codex/issue-320-analysis
Oct 19, 2026
Merged

CloudeaSoft merged 4 commits into
1.4.4from
codex/issue-320-analysis

Conversation

@CloudeaSoft

@CloudeaSoft CloudeaSoft commented Oct 17, 2026 •

Copy link
Copy Markdown
Member

Noise and heatmap textures have inconsistent names, duplicate copies across modules, and several files whose encoding does not match their extension. This PR cleans up that asset subset and updates its references.

Closes: #320 (noise and heatmap textures only).

Changes

  • Normalize 253 texture names using consistent casing and underscore word boundaries, preserving existing words, spelling, and numbers.
  • Move 7 shared textures into Function and remove 33 byte-identical duplicates. Update generated asset references and explicit or dynamically constructed paths.
  • Remove unused noise and heatmap parameter bindings and declarations from GrayFlow and the furnace lava-window shader. Shader sampling and color calculations are unchanged.
  • Re-encode 12 JPEG-backed .png files as PNG and 2 PNG-backed .bmp files as BMP. File paths, dimensions, and ImageSharp-decoded RGBA pixels are preserved.

Unused asset candidates are retained. The existing Acropora texture bindings are also preserved.

Validation

  • dotnet build /p:WarningLevel=0 passed with 0 errors and existing ImageSharp vulnerability warnings.
  • dotnet test --no-build --no-restore --verbosity normal /p:WarningLevel=0: 583 tests passed.
  • Verified exact-byte renames and duplicate removal, decoded pixel equality for format conversions, resource manifest paths, stale references, UTF-8 BOM checks, and git diff --check.
  • In-game rendering and world generation have not been exercised.

Standardize asset-name casing and underscore word separation while preserving technical acronyms, original words, image bytes, and directory locations. Update generated asset members and literal resource paths across active modules.
Consolidate identical noise and heatmap assets, move cross-module shared textures into Function, and update references. Preserve raw BMP resources needed by world generation alongside their texture-loading counterparts.
@CloudeaSoft CloudeaSoft changed the title Clean up noise and heatmap texture assets refactor(assets): clean up noise and heatmap textures Oct 17, 2026
@CloudeaSoft
CloudeaSoft enabled auto-merge (squash) October 17, 2026 06:18
@CloudeaSoft CloudeaSoft added this to the 1.0.0 - Yggdrasil Town milestone Oct 19, 2026
@CloudeaSoft CloudeaSoft self-assigned this Oct 19, 2026
@CloudeaSoft
CloudeaSoft merged commit 1c6a90e into 1.4.4 Oct 19, 2026
13 checks passed
@CloudeaSoft
CloudeaSoft deleted the codex/issue-320-analysis branch October 19, 2026 14:26
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.

Fix texture names in Everglow.Function and Everglow.Yggdrasil

2 participants