Expand EasingCurve
documentation
#17778
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Objective
EasingCurve
.Solution
Curve<T>
forEaseFunction
.EasingCurve::new(0.0, 1.0, EaseFunction::X)
can be shortened toEaseFunction::X
.Added some benchmarks for bonus points.Side Notes
EaseFunction
toEaseFn
for brevity, but that would be a breaking change and maybe controversial.EasingCurve
should beEaseCurve
, but say la vee.EaseFunction::Smoothstep
is still slower than callingsmoothstep
directly.EaseFunction::eval
.Testing
cargo test --package bevy_math cargo doc --package bevy_math ./target/doc/bevy_math/curve/easing/struct.EasingCurve.html cargo bench --package benches --bench math -- easing