Skip to content

Update monolith_inner.png - #4769

Open
XeKr wants to merge 1 commit into
Anvil-Dev:dev/1.21/1.6from
XeKr:assets/260909
Open

Update monolith_inner.png#4769
XeKr wants to merge 1 commit into
Anvil-Dev:dev/1.21/1.6from
XeKr:assets/260909

Conversation

@XeKr

@XeKr XeKr commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@Gugle2308

Copy link
Copy Markdown
Collaborator

All verification complete. This is a clean single-file texture PR by the original asset author (XeKr, whose Blockbench credit appears in the monolith models added in #4733). Writing the review:

代码审查摘要 — PR #4769

操作: opened
范围: 1 个文件 (0 Java, 0 新增, 0 删除) / 3 行 diff — 纯二进制纹理更新

📋 变更内容

项目 结果
文件 src/main/resources/assets/anvilcraft/textures/block/monolith_inner.png
blob 指纹 d9d7202bc5c4626e9a1a(与 diff index 行逐字一致,真实内容变更,md5 70efe28dd7b07b90
格式 16×16 索引色 PNG(IHDR 验证),新旧一致,无 .mcmeta(静态纹理无动画同步问题)
引用链 变更前后均被同一组 4 个模型引用:monolith_core/line + giant_monolith_core/line#1 槽位),无引用丢失、无孤儿风险
代码依赖 无 Java 侧引用;MonolithCoreBlockEntityRenderer 直接采样 vanilla 粒子纹理,不依赖本贴图像素

🎨 视觉变更(像素级验证)

PLTE 调色板实测(非目测推断):

  • 旧版:4 色暗红褐噪点(#221316/#3f191e/#2a1518/#34171a),柔和云状过渡
  • 新版:7 色暖金/赭石矿物质感(#5b291d/#8a4921/#d7a337/#e9c241/#c1822d/#773b1f…),带深褐斑点和亮金矿点,细节更丰富、可平铺性更好

该纹理映射到巨构框架的非 cullface 内部截面,与外壳 monolith.png(近黑炭灰 #0d0d0d 系)形成"黑壳 + 金芯"对比。

💡 观察(非阻塞)

  • 色调语义变化add monolith assets #4733 添加资产时内芯为"暗红"基调,本次改为"暖金"。黑壳配金内芯是否比暗红内芯更符合设计意图,属作者本人的美术决定(模型 credit 即为 XeKr),此处仅记录不干预。
  • 与 26.1 分支无同步需求(dev/26.1/1.6 上尚无此纹理路径,属 1.21 资产先行流程)。

🟢 看起来不错

  • 提交日期 2026-09-09,紧接 add monolith assets #4733 资产预置后的作者自查修正,改动精准单一,无生成数据漂移、无无关文件。
  • 真实增量与标题完全一致,无冗余重放。

结论: APPROVE — 单文件纹理更新,格式合法、引用链完整、blob 变更真实。PR 标题已准确描述变更内容,无需修改。

@Gugle2308

Copy link
Copy Markdown
Collaborator

💾 Self-improvement review: Patched SKILL.md in skill 'texture-asset-pr-review' (1 replacement).

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.

2 participants