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 documentation #37

Open
xdanielc opened this issue Aug 21, 2023 · 1 comment
Open

Improve documentation #37

xdanielc opened this issue Aug 21, 2023 · 1 comment

Comments

@xdanielc
Copy link

I have a few questions that I think should fall under the improve documentation category.

Can I also add non.static object in the master scene? On the sample scenes there's non-static objects on the main scene, so I guess you can put everything there, right?

I shouldn't bake lightning there either it either right? So the first one should be the fallback one? Should I select that in unity's official baked lightmaps or don't touch that?

If I have a mixed or baked light that is common to every scene, it should be on the master scene as well?

I think that's all for now, thank you for this tool!

@laurenth-personal
Copy link
Owner

laurenth-personal commented Aug 27, 2023

Hello,
Thank you for these questions. sorry for the delay, here are some answers :

  • You can add non static objects to your master scene, you just need to make sure they don't contribute to GI, or that they won't have an impact on GI that will be problematic for your lighting scenrios (light a door baked closed that you want to open and you want light to come in). For doors I recommend excluding them from GI
  • There is no default or fallback lighting scenario in this script, it doesn't do anything by default. Your scene will load the state that is saved, which is usually the last one you baked. The recommended way is to call the "load lighting scenario" on enable with the default lighting scenario you want to load.
  • Anything that is common to all scenarios can be indeed in the master scene, lights included indeed

Please don't hesitate to let me know if this is not clear or if you need more info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants