Skip to content

Intensity property for GLDEFS and A_AttachLights #3085

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

Merged
merged 1 commit into from
May 17, 2025

Conversation

nashmuhandes
Copy link
Contributor

  • Add "Intensity" property for dynamic lights in GLDEFS that will multiply the brightness of the light, useful for overbrightening/underbrightening a light.

  • Also add an "intensity" parameter for A_AttachLight in ZScript.

Note that for any kind of light overbrightening to do anything at all, one of the unclamped LightBlendModes in MAPINFO must be enabled.

Example files:

LightDefIntensityTest.zip
LightDefIntensityTest_3DWeapon.zip

One of the examples uses a 3D model weapon, the other does not. This is to demonstrate the lighting working on each situation.

Also demonstrated is a modified tech torch with the intensity intentionally cranked up via GLDEFS. And another modified red torch with its intensity dialed way down, again via GLDEFS.

Various Actors are placed in the map to demonstrate the effects of the intensity property working properly as far as Actor lighting is concerned.

Lastly, there is a flashlight created using A_AttachLight with an intentionally-cranked-up intensity value, again, to demonstrate the new feature.

…tiply the brightness of the light, useful for overbrightening/underbrightening a light.

- Also add an "intensity" parameter for A_AttachLight in ZScript.

Note that for any kind of light overbrightening to do anything at all, one of the unclamped LightBlendModes in MAPINFO must be enabled.
@nashmuhandes nashmuhandes requested a review from RicardoLuis0 May 16, 2025 18:01
@nashmuhandes nashmuhandes marked this pull request as ready for review May 16, 2025 20:41
@RicardoLuis0
Copy link
Collaborator

looks good 👍

@RicardoLuis0 RicardoLuis0 merged commit b44f8f0 into ZDoom:master May 17, 2025
9 checks passed
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.

2 participants