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 an option to toggle visuals in the moving navigation agents benchmarks #85

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

OverloadedOrama
Copy link
Contributor

@OverloadedOrama OverloadedOrama commented Jun 29, 2024

To make it easier to just check CPU usage of the navigation system itself, without visuals. The visuals are enabled by default, like they are in the physics benchmarks, but I can make them disabled if desired. EDIT: They are now off by default.

@Calinou
Copy link
Member

Calinou commented Jun 29, 2024

This should be applied to the physics benchmarks as well. I'd also suggest parsing a command-line argument instead of using a variable that needs to be changed manually in the source code, e.g. --debug-visuals.

There's a system parsing CLI arguments into member variables if they exist with a arg_ prefix in main.gd: https://github.com/godotengine/godot-benchmarks/blob/main/main.gd#L5-L10

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.

None yet

2 participants