Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve the 3D graphics settings demo #719

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Apr 8, 2022

Follow-up to #713.

  • New graphics settings:
    • Field of view.
    • Volumetric fog.
  • Use Godot's built-in 3D viewport facilities instead of using a SubViewport.
    • This also allows for choosing AMD FidelityFX Super Resolution for resolution scaling, but it's not working yet due to a Godot bug.
    • Nearest-neighbor scaling is no longer available. It can be reimplemented in Godot core and then exposed in the future.
  • Separate MSAA from FXAA settings, as both can be used at the same time.
  • Add more quality settings for various effect settings.
  • Rename sections and setting names to be more "player-oriented" (even if this results in the actual Godot terminology not being presented).
  • Add performance hints to setting values.
  • Add FPS counter and viewport resolution display (taking the resolution scale into account).
  • Add more objects to the test scene and tweak environment settings.
    • ACES tonemapping is now used.
    • Debanding is now enabled, as this scene requires it to avoid visible banding.
    • Make the emissive box less bright to prevent MSAA and FXAA from being ineffective on it (due to Godot not supporting correct HDR antialiasing).
    • Remove unnecessary DirectionalLight3D, VoxelGI and ReflectionProbe.

There's more stuff that could be done in the future (such as adding shadow quality settings), but this is a good start already.

Preview

2022-04-09_00 26 29

@Calinou Calinou force-pushed the improve-3d-graphics-settings-demo branch 5 times, most recently from 250d875 to 0e7dbed Compare April 9, 2022 00:11
@aaronfranke aaronfranke added this to the 4.0 milestone Apr 9, 2022
@aaronfranke aaronfranke self-requested a review April 9, 2022 00:30
@Calinou Calinou force-pushed the improve-3d-graphics-settings-demo branch 2 times, most recently from c05b202 to 5289438 Compare April 9, 2022 16:34
3d/graphics_settings/settings.gd Outdated Show resolved Hide resolved
3d/graphics_settings/settings.gd Outdated Show resolved Hide resolved
3d/graphics_settings/settings.gd Outdated Show resolved Hide resolved
- New graphics settings:
  - Field of view.
  - Volumetric fog.
- Use Godot's built-in 3D viewport facilities instead of using a SubViewport.
  - This also allows for choosing AMD FidelityFX Super Resolution
    for resolution scaling, but it's not working yet due to a Godot bug.
  - Nearest-neighbor scaling is no longer available. It can be
    reimplemented in Godot core and then exposed in the future.
- Separate MSAA from FXAA settings, as both can be used at the same time.
- Add more quality settings for various effect settings.
- Rename sections and setting names to be more "player-oriented"
  (even if this results in the actual Godot terminology not being
  presented).
- Add performance hints to setting values.
- Add FPS counter and viewport resolution display
  (taking the resolution scale into account).
- Add more objects to the test scene and tweak environment settings.
  - ACES tonemapping is now used.
  - Debanding is now enabled, as this scene requires it to avoid visible
    banding.
  - Make the emissive box less bright to prevent MSAA and FXAA from
    being ineffective on it (due to Godot not supporting correct
    HDR antialiasing).
  - Remove unnecessary DirectionalLight3D, VoxelGI and ReflectionProbe.
@Calinou Calinou force-pushed the improve-3d-graphics-settings-demo branch from 5289438 to 48d2ec9 Compare April 25, 2022 18:06
@aaronfranke aaronfranke merged commit 5c40412 into godotengine:4.0-dev Apr 25, 2022
@Calinou Calinou deleted the improve-3d-graphics-settings-demo branch April 25, 2022 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants