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

Add access to interpolation fraction for fixed timestep interpolation #30226

Merged
merged 1 commit into from
Jul 18, 2019

Conversation

lawnjelly
Copy link
Member

Addresses #30068

This is a prerequisite for allowing proper support for fixed timestep interpolation, exposing the interpolation fraction to the engine, modules and gdscript.

The interpolation fraction is the fraction through the current physics tick at the time of the current frame.

Note: I hope I have the format correct for putting the function in the docs, I haven't done this before.

Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, please take a look at the comments I left 😃

core/engine.h Outdated Show resolved Hide resolved
doc/classes/Engine.xml Outdated Show resolved Hide resolved
Addresses godotengine#30068

This is a prerequisite for allowing proper support for fixed timestep interpolation, exposing the interpolation fraction to the engine, modules and gdscript.

The interpolation fraction is the fraction through the current physics tick at the time of the current frame.
@lawnjelly
Copy link
Member Author

All changes requested have been done, I've also run the doctool and moved the help xml item into alphabetical order. Should be good to go now.

Just to make clear it shouldn't affect any engine functionality, just exposes an extra variable. Smoothing nodes are nearly ready for testing as a module (both 3d and 2d versions) once this is merged.

@akien-mga akien-mga merged commit 20a3bb9 into godotengine:master Jul 18, 2019
@akien-mga
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants