-
-
Notifications
You must be signed in to change notification settings - Fork 97
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
Expose World3D.compositor
#9815
Comments
This is not a game problem. What is the problem you are solving in your game? Some guesses from my clients in Game Development:
|
@fire i have not the slightest clue what your example is about, but that really doesn't sound related to and the problem is that i want to access so yes, "the problem or limitation" i have in my project that is about managing the should this ticket just be a bug report maybe? pretty sure that just got missed in the PR, there was no mention about its omission as far as i could tell. |
I have no clue why you would want to use World3D and its compositor property so I'm trying to figure out. Assuming it's used for something. Saying that World3D and its compositor property is useful because it's the World3D and compositor makes me confused and lost. Edited: Trying to figure out why https://github.com/godotengine/godot/blob/master/scene/resources/3d/world_3d.cpp#L151-L152 get_environment is not in your build. |
@fire yeah i wasnt aware this issue is supposed to document existing features just for mentioning them. usually people look things up in docs or ask if they don't know what a builtin class does before blaming people for being "unclear what it's even for" and "not describing 'a game problem'". because sometimes the problem really is "someone forgot this", and not just "i have no idea what I'm even doing". this isn't an "X/Y problem", it's "trying to access the 'Y' that exists for the sole purpose of enabling 'X', but directly instead of abusing a Node or circumventing the whole Viewport", just you apparently didn't know that "Y" even existed before and gdscript/classdb/docgen doesn't get to see it (for no reason).
and get/set (plus the actual property decl for) |
Describe the project you are working on
a script (planned addon) to manage WorldEnvironment and Camera related effects
Describe the problem or limitation you are having in your project
World3D
exposesenvironment
andcamera_attributes
and does have but does not expose thecompositor
property.Describe the feature / enhancement and how it helps to overcome the problem or limitation
it would allow accessing the
compositor
property right on theWorld3D
, where it "matters", instead of having to go through a "dummy" WorldEnvironment node's internal setter or the Server.Describe how your proposal will work, with code, pseudo-code, mock-ups, and/or diagrams
by being the
World3D.compositor
it already is, but visible through the ClassDBIf this enhancement will not be used often, can it be worked around with a few lines of script?
it can be worked around by adding a
WorldEnvironment
node and setting the property on it instead (which will then callWorld3D.set_compositor
internally on the world of its associated viewport).it can also be worked around by calling
RenderingServer.scenario_set_compositor
, but that won't actually update the World3D, just work all the way around the scenetree/viewport.neither of them allow the
World3D
resource to retain itscompositor
property in storage.Is there a reason why this should be core and not an add-on in the asset library?
because that's where it already is, we just miss 3 lines of telling the ClassDB about it (and its accessors, that's the other 2 lines).
The text was updated successfully, but these errors were encountered: