-
Notifications
You must be signed in to change notification settings - Fork 12
Boids Demo #171
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
Boids Demo #171
Conversation
…oving into different directions
…mplemented. Major refactor, Godot implementation broken for now
functions
|
Hi! Sorry for getting back so late. I'd love to merge; however you appear to have quite a lot of changes unrelated to the boids demo in your PR. Do you know how to approach this? I'm happy to lend a hand if you run into trouble :) |
functions
|
Thanks for the feedback! EDIT: Actually I managed to revert the changes in the files out of scope, or at least these changes are not shown as changed in this PR anymore. I tried to undo the changes that happened to the files out of scope, but had no luck so far. Afaik the out of scope changes were only commited in this commit: 76567b9. Any help or advice to just revert the changes for these files out of scope would be highly appreciated! You can also contact versjon or me (maxk_2) in Discord. |
Ivorforce
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome. Thank you both again @versjon and @kro-ma for your hard work!
I will merge this PR now, and likely have a look over later. I recall you saying the Godot version is currently faster than the NumDot version, which may make it a great opportunity to improve performance internally :)
If you find anything else you want to improve, feel free to make a follow-up PR!
Added boids demo with gdscript and numdot solver, currently numdot version less performant, #48