Skip to content

Brighten default material previews and refresh documentation - #851

Merged
yuecideng merged 1 commit into
mainfrom
codex/brighten-material-previews
Oct 24, 2026
Merged

yuecideng merged 1 commit into
mainfrom
codex/brighten-material-previews

Conversation

@yuecideng

Copy link
Copy Markdown
Contributor

Description

The material catalog thumbnails were too dark to show surface texture and brushed-metal detail clearly. Apply the approved brighter studio setup to preview-material and replace all 24 individual documentation images plus four galleries with the approved OfflineRT renders.

  • Use linear exposure 1.6, environment intensity 48, key light 150 and fill light 120. Material parameters, camera poses and preview geometry stay consistent with the existing catalog.
  • Identify the updated scene as material-studio-v2 and record its lighting values in results.json for reproduction.
  • Document the studio recipe and capture settings in the material catalog and CLI guide. The 28 WebPs total approximately 1.23 MB and remain excluded from the wheel.

No dependency changes.

Type of change

  • Enhancement
  • Documentation update

Validation

  • 26 existing preview CLI tests passed. Existing coverage was reused; no tests duplicating lighting constants were added.
  • Production CLI regenerated all 24 OfflineRT materials at 640×640, six accumulation frames and 12 render iterations. Every capture passed direct-native comparison; maximum center-crop RMSE was 0.174/255. The production captures reproduced the approved previews with zero full-frame RMSE.
  • All four compatible built-ins passed native comparisons in both Hybrid and FastRT at 192×192, with zero RMSE.
  • Material/CLI HTML guides build with three unchanged ReuseSegmentState duplicate-attribute warnings. Updated recipe text and image assets were checked in the built output.
  • black==26.3.1: black .; API docs 3,042/3,042; context check and git diff --check pass.

Context review: existing material-context guidance already assigns studio settings to the preview command and links the user guides. No context edit is needed for the illumination adjustment.

Screenshots

Previous lighting Approved brighter lighting
Previous built-in material gallery Brighter built-in material gallery

All 24 updated material previews

Checklist

  • Ran black ..
  • Updated affected documentation and reviewed the owning project context.
  • Public API documentation coverage passes; exported APIs are unchanged.
  • Reused relevant behavioral coverage and qualified actual native renders.
  • Dependencies are unchanged.

@yuecideng yuecideng added docs Documentation, API references and example explanations. enhancement Improvements to existing capabilities; use feature for new capabilities. rendering Image generation, lighting, material rendering, native renderers and image quality. labels Oct 24, 2026
@yuecideng
yuecideng merged commit 54870f2 into main Oct 24, 2026
5 of 8 checks passed
@yuecideng
yuecideng deleted the codex/brighten-material-previews branch October 24, 2026 07:05
@greptile-apps

greptile-apps Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable issues were found.

Summary

Brightens material previews with shared studio settings and records them in results.json.

  • Material previews now use brighter studio lighting.

Reviews (1) · Last reviewed commit: "docs(rendering): brighten default materi..." · Reviewed by Greptile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation, API references and example explanations. enhancement Improvements to existing capabilities; use feature for new capabilities. rendering Image generation, lighting, material rendering, native renderers and image quality.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant