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

Decide if enabling physics interpolation is the right thing to do #3679

Closed
hhyyrylainen opened this issue Aug 10, 2022 · 5 comments · Fixed by #4521
Closed

Decide if enabling physics interpolation is the right thing to do #3679

hhyyrylainen opened this issue Aug 10, 2022 · 5 comments · Fixed by #4521
Labels
intermediate programming review
Projects
Thrive Planning
Milestone
Release 0.6.4

Comments

@hhyyrylainen
Copy link
Member

Godot 3.5 adds physics interpolation that can be enabled. We might get some benefit from enabling that.
@hhyyrylainen hhyyrylainen added this to Backburner in Thrive Planning via automation Aug 10, 2022
@Zireael07
Copy link

Considering you had some physics performance issues, I'd try reducing physics fps and turning interpolation on ;)

@hhyyrylainen
Copy link
Member Author

I'm not actually convinced our problem is the physics performance. It seems in all kinds of profiling the physics processing time doesn't really go up at all, really... though to conclusively say someone could turn interpolation on and reduce the physics framerate to see if it actually would have an effect, but my guess is it wouldn't have more than a few percent effect.

@hhyyrylainen hhyyrylainen moved this from Backburner to In progress in Thrive Planning Oct 13, 2023
@hhyyrylainen hhyyrylainen added this to the Release 0.6.4 milestone Oct 13, 2023
@hhyyrylainen hhyyrylainen mentioned this issue Oct 13, 2023
Merged
4 tasks
Thrive Planning automation moved this from In progress to Done Oct 27, 2023
@hhyyrylainen
Copy link
Member Author

Now with the Jolt physics it would need to first be investigated if this is even possible to do anymore.

@Zireael07
Copy link

I must have missed where you switched to Jolt...

@hhyyrylainen
Copy link
Member Author

It was just merged yesterday: #4521 (shows up above as an event on this issue being closed by that).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
intermediate programming review
Projects
Thrive Planning
  
Done
Milestone
Release 0.6.4
2 participants
@hhyyrylainen @Zireael07