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.

Sign up for GitHub

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

Jump to bottom

document NavMeshQuery random seed functionality #409

Open
GGAlanSmithee opened this issue Jul 9, 2024 · 2 comments
Open

document NavMeshQuery random seed functionality #409

GGAlanSmithee opened this issue Jul 9, 2024 · 2 comments

Comments

@GGAlanSmithee
Copy link
Contributor

I recently created a new project with an updated version of recast-navigation, and noticed the deterministic nature of NavMeshQuery.findRandomSeed, which is amazing.

Even though the implementation makes perfect sense, I was still a bit surprised to see that I got the same series of reproducable randomness as I ran my project over and over, so I was wondering

  • is this how it used to be before, but implicit, or is this a breaking change from before you introduced setRandomSeed?
  • does the documentation mention this anywhere?

For the second question, I could not find any documentation on it, and wanted to add it to the README, but could not find a good place to put it. I think it would make sense to mention this functionality, the default seed of 1337, and the expected behavior.
@isaac-mason
Copy link
Owner

Hey @GGAlanSmithee!

Thanks for creating the issue. There isn't any documentation for this outside of the auto generated docs yet. I'll add some documentation for randomness & setting the random seed shortly and share it here to get your thoughts.

This isn't a breaking change - previously the same seeded random number generator was already being used, but now the seed can be set.

@GGAlanSmithee
Copy link
Contributor Author

@isaac-mason

cool!

This isn't a breaking change - previously the same seeded random number generator was already being used, but now the seed can be set.

Yeah, I figured. This is really nice though!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@GGAlanSmithee @isaac-mason