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

Add apparent emitter brightness controls #1555

Closed

Conversation

Peregrine05
Copy link
Member

@Peregrine05 Peregrine05 commented Feb 28, 2023

  • Renamed Emitter intensity to Emitter brightness, and changed it to scale both emitter light intensity and apparent emitter brightness. Changed its default value to 1.
  • Added Emitter light intensity modifier, which scales only the emitter light intensity. Its default value is 13, to reflect the default value of Emitter intensity from previous versions.
  • Added Apparent emitter brightness modifier, which scales only the apparent brighness of emitters. Its default value is 1.
  • Added an Apparent brightness material property, which changes the apparent brightness of the material when it behaves as an emitter. Its default value is 1
  • When loading scenes saved in an older Chunky version, Chunky will load the value of Emitter brightness (previously Emitter intensity) as the value of Emitter light intensity modifier, and set Emitter brightness to 1.

New:
image
image

This PR could still use some testing, specifically with ESS. The PathTracer code might also be more complicated than it need be; we should check if there is any way to achieve the same results with simpler code (This may no longer be the case).

JustinTimeCuber and others added 10 commits February 16, 2023 23:28
It was accidentally removed.
- Change `Emitter intensity` to scale both emitter light intensity and apparent emitter brightness.
- Add `Emitter light intensity`, which scales only the emitter light intensity.
"looks like firstReflection is only used for fog, but yeah don't want to mess with our jank xd" (NotStirred, [Discord](https://discord.com/channels/541221265512464394/545374333883777037/1079435203879129220))

Co-Authored-By: Tom Martin <[email protected]>
@Peregrine05 Peregrine05 changed the title Apparent emitter brightness Add apparent emitter brightness controls Feb 28, 2023
@Peregrine05 Peregrine05 marked this pull request as ready for review March 16, 2023 18:59
@Peregrine05 Peregrine05 marked this pull request as draft March 16, 2023 23:05
@Peregrine05 Peregrine05 marked this pull request as ready for review March 17, 2023 23:04
Peregrine05 and others added 6 commits March 22, 2023 20:04
`1/35` is approximately equal to 0.029; 0.02 would make soul torches appear too dim.
Make the per-material apparent brightness sliders behave similarly to the global ones
@Peregrine05
Copy link
Member Author

Fixed by #1627 and #1621.

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.

3 participants