Skip to content

Turn off physics and 3d navigation servers in the Project Manager. - #54422

Merged
akien-mga merged 1 commit into
godotengine:masterfrom
ibrahn:project-manager-nophys
Nov 16, 2021
Merged

Turn off physics and 3d navigation servers in the Project Manager.#54422
akien-mga merged 1 commit into
godotengine:masterfrom
ibrahn:project-manager-nophys

Conversation

@ibrahn

@ibrahn ibrahn commented Oct 30, 2021

Copy link
Copy Markdown
Contributor

Copying the editor behaviour of deactivating these servers we're not
using, to reduce CPU load.

Partial fix for #54291

Copying the editor behaviour of deactivating these servers we're not
using, to reduce CPU load.
@Calinou

Calinou commented Oct 30, 2021

Copy link
Copy Markdown
Member

See also #38208.

Out of curiosity, why can't NavigationServer2D be disabled? I don't see a set_active() method for it, unlike NavigationServer3D.

@ibrahn

ibrahn commented Oct 30, 2021

Copy link
Copy Markdown
Contributor Author

Editor lines, just for easy reference:

godot/editor/editor_node.cpp

Lines 5782 to 5788 in 6b0b1a4

NavigationServer3D::get_singleton()->set_active(false); // no nav by default if editor
PhysicsServer3D::get_singleton()->set_active(false); // no physics by default if editor
PhysicsServer2D::get_singleton()->set_active(false); // no physics by default if editor
ScriptServer::set_scripting_enabled(false); // no scripting by default if editor

@ibrahn

ibrahn commented Oct 30, 2021

Copy link
Copy Markdown
Contributor Author

I wondered about the Nav2D server as well, but I couldn't find any cases where we deactivate that one, or how to do it, so it may be a more involved puzzle. Neither of the Nav servers seemed to be much of a factor in the CPU usage though so it might not be much of a priority here.

@akien-mga
akien-mga merged commit bef20f5 into godotengine:master Nov 16, 2021
@akien-mga

Copy link
Copy Markdown
Member

Thanks!

@akien-mga akien-mga added the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Nov 16, 2021
@akien-mga

Copy link
Copy Markdown
Member

Cherry-picked for 3.5.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Nov 18, 2021
@ibrahn
ibrahn deleted the project-manager-nophys branch April 7, 2025 09:07
BendyLand pushed a commit to BendyLand/voltaire that referenced this pull request Aug 2, 2026
BendyLand pushed a commit to BendyLand/voltaire that referenced this pull request Aug 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants