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

Stuttering motion with multiple characters with 'stray' behavior #15

Closed
AndyTechGuy opened this issue Jun 17, 2019 · 2 comments

Comments

Projects
None yet
3 participants
@AndyTechGuy
Copy link

commented Jun 17, 2019

With multiple characters using the same stray behavior, a target update on one character updates the targets of all stray characters. This can be seen in the gif below, with insensitiveDeer characters from Terasology/WildAnimals:

deerMovement

The stray behavior works perfectly with only one character, but with more and more characters the movement becomes more and more erratic.

@dkambersky dkambersky self-assigned this Jun 18, 2019

@casals casals self-assigned this Jun 19, 2019

@dkambersky

This comment has been minimized.

Copy link
Contributor

commented Jun 19, 2019

Update: I had a hunch and now am pretty sure that this is a duplicate / symptom of #6 . Still looking into it. Ugly but, looking through a lot of levels of abstraction / delegation which makes stuff pretty hard to debug.

One thing you can't say is that Synopia underengineered things :P

@dkambersky

This comment has been minimized.

Copy link
Contributor

commented Jun 19, 2019

So while this one is fixed (I had it fixed for ages in my version, just for some reason it never made its way upstream), the underlying issue likely isn't. Let me know in #6 if you find any other interesting ways to reproduce it. ( @casals you said you ran into this at some point too?)

tl;dr the outer 'loop' node is useless and breaks things, don't use it. Still no reason it should break things this way though, so this isn't entirely fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.