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

Defunct graphics settings #5152

Open
WeDoStuffUncorp opened this issue Oct 22, 2023 · 5 comments
Open

Defunct graphics settings #5152

WeDoStuffUncorp opened this issue Oct 22, 2023 · 5 comments
Labels
Revive: Keep issue has been looked at and deemed potentially helpful for revive Status: Needs Investigation Requires to be debugged or checked for feasibility, etc. Topic: Stabilization Requests, Issues and Changes related to improving stablity and reducing flakyness Type: Bug Issues reporting and PRs fixing problems

Comments

@WeDoStuffUncorp
Copy link

WeDoStuffUncorp commented Oct 22, 2023

While testing Terasology's graphics settings recently, I found a few settings that didn't seem to work as intended or at all.

The following is a list of all of these settings, as well as notes on each one:

  • Clamp Lighting - no notes
  • Extra Lighting - as far as I can tell, it doesn't seem to actually add any extra lighting... may require further testing
  • ✔️ Vignette - makes the screen slightly darker at the edges, but the effect isn't noticeable enough to bring any value
  • ✔️ Eye-Adaptation - like vignette, the effect of the setting isn't noticeable enough, but I believe that if it can be upscaled then it could be useful
  • ✔️ Bloom - doesn't seem to work, and strangely disables the Light Shafts setting while it's activated
@WeDoStuffUncorp WeDoStuffUncorp added Status: Needs Investigation Requires to be debugged or checked for feasibility, etc. Topic: Stabilization Requests, Issues and Changes related to improving stablity and reducing flakyness Type: Bug Issues reporting and PRs fixing problems labels Oct 22, 2023
@jdrueckert jdrueckert added this to the 2023 Revive - Milestone 3 milestone Oct 22, 2023
@jdrueckert jdrueckert added the Revive: Keep issue has been looked at and deemed potentially helpful for revive label Oct 22, 2023
@AuroraBorealish
Copy link

Additional graphical issue: Torches all flicker in unison with "Flickering lights" on, which is the default setting.
Personally, this is quite jarring, and makes it so that no matter how many torches, it will be completely dark every 2 seconds or so, when inside a house with indoor lighting.

@Imitater967
Copy link
Contributor

Imitater967 commented Mar 25, 2024

About Vignette, we need to change the texture mannually at org/terasology/engine/assets/textures/vignette.png
About the eye adaption, I already fixed it at https://github.com/Terasology/CoreRendering/pull/77
About the Bloom, exactly what you said.
ExtraLightning is normalMapping, but we don't see any normalmap related document in the tutorial.
ClampLightning is just a empy property and just usless,
so I suggest remove clamplightning in both ui and Rendering Confing,
bloom, extralightning just remove them in UI, because they are likely needed in the future.
image

@Imitater967
Copy link
Contributor

Imitater967 commented Mar 25, 2024

Additional graphical issue: Torches all flicker in unison with "Flickering lights" on, which is the default setting. Personally, this is quite jarring, and makes it so that no matter how many torches, it will be completely dark every 2 seconds or so, when inside a house with indoor lighting.

You can find related code at globalFunctionsFragIncl.glsl, the flickeringLightOffset seems separated perchunk, instead of perblock.

I think, we may add some parameter like waving the water .

forexample
image

@Imitater967
Copy link
Contributor

2 quest remain.
remove clamp lightning from UI, rendering config and any related code you can find.
investigate how to separate the flickering light offset per block instead of per chunk (we need to be considerate of the performance impact here, though - if it impacts performance noticably we probably don't want to merge that until we improved performance a bunch first)

@jdrueckert
Copy link
Member

Eye-adaptation fixed by Terasology/CoreRendering#77
Bloom fixed by Terasology/CoreRendering#78
Vignette improved by Terasology/CoreRendering#80

Thank you for the great work @Imitater967 💚

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Revive: Keep issue has been looked at and deemed potentially helpful for revive Status: Needs Investigation Requires to be debugged or checked for feasibility, etc. Topic: Stabilization Requests, Issues and Changes related to improving stablity and reducing flakyness Type: Bug Issues reporting and PRs fixing problems
Projects
None yet
Development

No branches or pull requests

4 participants