Details

Reviewers

Campbell Barton (campbellbarton)
Dalai Felinto (dfelinto)
Mitchell Stokes (moguri)

Commits

rBS5d0696052a00: BGE: Fix T41502 Path following jumping
rB5d0696052a00: BGE: Fix T41502 Path following jumping

Summary

New Lock Z velocity parameter was added. This parameter avoid the micro-jumping.
By default it is actived except when you load an old file that it is deactived to keep former behaviour.

Additionally it was solved another issue related with the acceleration: That is the acceleration value was not taked into account and we had always the maximum linear velocity from the beginning of movement. Now the acceleration is taken into account until we reach the maximum velocity.
When you load an old file, the acceleration value is set to the maximum range (1000.f). This way we simulate a maximum velocity constant from the beginning of movement (former behaviour).

Diff Detail

Repository: rB Blender
Branch: bge-bug-path-finding

Event Timeline

Jorge Bernal (lordloki) updated this revision to Diff 3391.Feb 7 2015, 3:38 AM

Jorge Bernal (lordloki) retitled this revision from to BGE: Fix T41502 Path following jumping.

Jorge Bernal (lordloki) added reviewers: Mitchell Stokes (moguri), Dalai Felinto (dfelinto), Campbell Barton (campbellbarton).

Jorge Bernal (lordloki) updated this object.

I'm not totally convinced with this solution. I'm investigation a more complete solution that takes care the other issues in the steering/path following actuator as acceleration is not taking into account, there is no curved movement in the turns, etc.

New Lock Z velocity parameter was added. This parameter avoid the micro-jumping. By default it is actived and when you load an old file is deactived to keep former behaviour.

Additionally it was solved another issue related with the acceleration: That is the acceleration value was not taked into account and we had always the maximum linear velocity from the beginning of movement.

Now the acceleration is taken into account until we reach the maximum velocity. When you load an old file, the acceleration value is set to the maximum range (1000.f). This way we simulate a maximum velocity constant from the beginning of movement (former behaviour).

Jorge Bernal (lordloki) updated this object.Feb 10 2015, 1:35 AM

Sergey Sharybin (sergey) added a subscriber: Sergey Sharybin (sergey).Feb 11 2015, 5:56 PM

Sergey Sharybin (sergey) added inline comments.

source/blender/blenloader/intern/versioning_270.c
634	No need to bump subversion yet. Just use `DNA_struct_elem_find` for until next bump is actually needed. P.S. Rule of thumb is: subversion bump is only done when major changes to DNA are made.