Skip to content

Conversation

@adriengivry
Copy link
Member

@adriengivry adriengivry commented May 14, 2025

Description

  • Updated default scene to include:
    • Post Process Stack
    • Skysphere
  • Added a default HDRi texture and a material for the skysphere
  • Fixed HDR textures were loaded as RGBA16F instead of RGBA32F
  • Updated grid to fade with horizon
  • Updated bloom effect default threshold from 0.8f to 1.0f
  • Updated ambient lights default values from 0.1f to 0.03f (works better with PBR)
  • Updated default material to use the StandardPBR.ovfx shader instead of StandardPBR.ovfx
  • Shadows are now enabled by default on directional lights
  • Updated scene view and asset view background color to full black (0,0,0) (not visible when using a skysphere)
  • Fixed asset import doesn't offer importing HDR by default
  • Updated CreateNewMaterial function used by the asset browser to not use hard coded XML materials, but use the serialization system instead.
  • Removed unused stencil usage in Grid
  • Added stencil clear to DebugActorRenderPass for outline rendering (was necessary since the clear was removed from the GridRendererPass)

Limitations

  • Bloom, when applied to skysphere, might not be visually accurate in some scenarios (bleeding)

Related Issue(s)

Fixes #380

To-Do

  • Fix bloom not working well with skysphere (bloom shouldn't apply to the skysphere at all) bloom should apply to the skysphere, but bloom settings could be tweaked by the user based on the skysphere.

Review Guidance

Write here.

Screenshots/GIFs

OvEditor_gXUT9p1HEC.mp4

_New flow from starting the editor

@adriengivry adriengivry self-assigned this May 14, 2025
@adriengivry adriengivry added Editor Something relative with the editor Graphics Graphical feature labels May 14, 2025
@adriengivry adriengivry mentioned this pull request May 14, 2025
1 task
@adriengivry adriengivry changed the title New default scene Improved default scene: skysphere + post-processing + better grid May 14, 2025
@adriengivry adriengivry marked this pull request as ready for review May 14, 2025 15:17
@adriengivry adriengivry merged commit 64c8f42 into main May 14, 2025
1 check passed
@adriengivry adriengivry deleted the feature/new_default_scene branch May 14, 2025 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Editor Something relative with the editor Graphics Graphical feature

Development

Successfully merging this pull request may close these issues.

Entities with alpha blending prevents the grid from rendering behind them

2 participants