PhysicalAddons / physical-starlight-and-atmosphere

issue tracker
16 stars 2 forks source link

Can't use the Addon on more than one Scene #26

Closed karlisup closed 3 years ago

karlisup commented 4 years ago

Describe the bug

A Background scene for only the atmosphere and a Foreground scene with objects that need to have a transparent background but receive the light from the addon.

When the addon gets enabled on the Background scene and then on the Foreground scene with transparency checked in the Film tab the Addon node disappears from the Background scene.

I now have to use a workaround with a holdout plane to eliminate the atmosphere to come through on some layers and have to use the Add blend mode instead of alpha over to merge the layers. Still not ideal.

Possible workaround (?): I copy and pasted the Atmospheric node from one scene to another and didn't touch the button to enable/disable the addon. It now works as expected.

Description copied and a bit adapted from this issue.

When new Scene created such error in console is returned: image

General information

To Reproduce

  1. Open a new project
  2. Enable PSA addon
  3. In topbar (or any other possible way) create a new scene

Screenshots image

karlisup commented 4 years ago

Hi! If somebody is using Scenes in Blender and can explain to me how would you expect this addon to work with scenes I would appreciate that. Also, I don't know how commonly people use this feature and would appreciate it if you could upvote the issue. It would help me prioritize feature development better.

LennartSolik commented 4 years ago

Well my problem and the reason i reported the issue was that i needed the athmosphere in 2 scenes (one with only the athmosphere and one with the model and tranparent background but still lit by the athmosphere addon) and that the toggle on/off button removed the node in the world editor from the "not active scene". So it was just possible to have one active athmosphere in your whole Blender project.

To fix the removal of the node in the "not active scene" i set the athmosphere to desired liking and then copied the node in the world shader editor to the other scenes world shader editor and never touched the on/off toggle again. After this i was able to control, for example the time of day and it would update on the second scene also.

It could be also really cool to have 2 scenes with 2 different athmospheres. As it is now its possible to control both nodes from one scene but enabling/disabling the addon removes one node from the world shader editor from the not active scene.

Maybe a toggle button to "link" settings from one scene to the athmosphere world node from another scene. And a toggle to "unlink" the settings so you can control 2 or even more different athmospheres in additional scenes.

ZanQdo commented 3 years ago

I would propose to move the properties storage to the World datablock instead of the Scene datablock. This way just by linking or copying the world, you get all the Physical Sky properties.

The addon's UI is even already inside the world window but the actual properties are under the scene. This could be changed for consistency and easy of use between different scenes.

image

karlisup commented 3 years ago

This bug was fixed in v1.4 early stage in a way we thought was right. With a help of @ZanQdo it was clear that it wasn't the right way to go and this has to be redone. Variables must be stored in a World data block instead of the scene and that would solve also issues with working with the scenes. As scenes are not causing any more console errors I would like to close this issue, just for the sake of keeping the bug tracker clean.