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

Linear interpolation for Integer parameter is not really linear #828

Closed
morevnaproject opened this issue May 15, 2019 · 3 comments
Closed

Linear interpolation for Integer parameter is not really linear #828

morevnaproject opened this issue May 15, 2019 · 3 comments
Projects
Milestone

Comments

@morevnaproject
Copy link
Member

@morevnaproject morevnaproject commented May 15, 2019

Synfig version & platform:
Latest development version, master branch, commit 3234bc0
(probably stable version affected too)

Issue description:
I have created a Star Layer.
It has parameter called "Points", which is an integer value type.
I have animated this parameter to change from 5 to 13 and set interpolation to Linear.
Then I looked at Graphs panel and found that the change of the parameter is not really linear. See screenshot:
screenshot_004

Source file -
star-ponts-2.sifz.zip

@morevnaproject

This comment has been minimized.

Copy link
Member Author

@morevnaproject morevnaproject commented May 15, 2019

@morevnaproject

This comment has been minimized.

Copy link
Member Author

@morevnaproject morevnaproject commented Jul 20, 2019

@blackwarthog I wonder, how long we had this issue with linear interpolation? If it was existing for older versions, then we probably need to bump canvas version and make sure the interpolation will work old way with old files. Otherwise the animation might get broken in some cases.

@morevnaproject

This comment has been minimized.

Copy link
Member Author

@morevnaproject morevnaproject commented Jul 22, 2019

After a quick discussion with @blackwarthog we've come to conclusion that compatibility here is not easy to handle and it actually not affects cases that I initially thought about.

@morevnaproject morevnaproject added this to Done in Releases Jul 29, 2019
@morevnaproject morevnaproject moved this from Done to Released in Releases Feb 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Releases
Released
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.