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

feat(flickerlight): add block offset for flick light #81

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

Imitater967
Copy link
Contributor

What i did

I added vertworld offset, at chunk_vert.glsl, inorder to make light flicker at different time.

How to test

dig a room deep under ground, place torches everywhere

@jdrueckert
Copy link
Member

jdrueckert commented Apr 7, 2024

Tested it out and I have a bit mixed feelings about this.

It definitely gets rid of the previous feel of "every light source pulses at the same point in time", but now it feels wavy... a bit like we're underwater and the shadows it creates on the floor don't make sense. It seems like there's a flow from one side to the other, not from the center out as you would expect from a light source 🤔
I somehow like the effect, but it is quite different from what we had before and from what I had expected...

Definitely need a second opinion here.

@jdrueckert
Copy link
Member

We discussed this in the last sync and agreed that while it would make an awesome underwater effect, it doesn't really fit the flickering light of a torch.
So I would propose that you either replace the effect with something more suitable, or re-purpose this PR to use it as an underwater effect and create a new one for the torch flicker @Imitater967 .

@Imitater967
Copy link
Contributor Author

Imitater967 commented Apr 14, 2024 via email

@jdrueckert
Copy link
Member

the water wave?

If with "water wave" you're referencing the effect you implemented in this PR, then yes, that's what I'm talking about. It would be a nice light effect on underwater surface blocks (the bottom of a river/lake/ocean)
If with "water wave" you're referencing the wave movement of the water in Terasology, then no, that's not what I mean.

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.

None yet

3 participants