Skip to content

Conversation

ghost
Copy link

@ghost ghost commented Apr 12, 2021

  • Setting the attribute affects camera smoothing and model position interpolation.
  • Adds 'decayMilliseconds' to Damper.update() method call.
  • Updates to docs - adds interpolation-decay
  • Updates to the Interpolation example - adds interpolation-decay attribute example

Reference Issue

Fixes #580

* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
@ghost ghost requested a review from elalish April 12, 2021 23:08
@elalish
Copy link
Contributor

elalish commented Apr 15, 2021

Also, please mark Fixes ... in the description so that Github will automatically link and close the issue when this is merged.

…interpolation-decay-api

Adds 'interpolation-decay-milliseconds' attribute
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
…interpolation-decay-api

Adds 'interpolation-decay-milliseconds' attribute
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
Fixes #580
@ghost ghost requested a review from elalish April 15, 2021 22:58
@ghost ghost changed the title Adds 'interpolation-decay-milliseconds' attribute Adds 'interpolation-decay' attribute Apr 15, 2021
Copy link
Contributor

@elalish elalish left a comment

Choose a reason for hiding this comment

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

This looks great, thanks! Let's add the documentation and then we can merge it.

…interpolation-decay-api

Adds 'interpolation-decay-milliseconds' attribute
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
* Updates to docs
Fixes #580
…interpolation-decay-api

Adds 'interpolation-decay-milliseconds' attribute
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
* Updates to docs
Fixes #580
@ghost ghost requested a review from elalish April 15, 2021 23:37
…interpolation-decay-api

Adds 'interpolation-decay-milliseconds' attribute
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
* Updates to docs
Fixes #580
@ghost ghost marked this pull request as ready for review April 16, 2021 02:10
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
* Updates to docs
* Updates to Interpolation example
Fixes #580
@ghost ghost requested a review from elalish April 16, 2021 18:43
* Setting the attribute affects camera smoothing and model position interpolation.
* Adds 'decayMilliseconds' to Damper.update() method call.
* Updates to docs
* Updates to Interpolation example
Fixes #580
@ghost ghost requested a review from elalish April 16, 2021 22:32
elalish
elalish previously approved these changes Apr 16, 2021
Copy link
Contributor

@elalish elalish left a comment

Choose a reason for hiding this comment

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

Thanks, this looks great!

* hide rendering while new model loads

* removed pointless test, minor cleanup

* skipping Edge flakes temporarily
@ghost ghost requested a review from elalish April 20, 2021 19:03
Copy link
Contributor

@elalish elalish left a comment

Choose a reason for hiding this comment

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

Thanks!

@ghost ghost merged commit f9e1f71 into master Apr 20, 2021
@ghost ghost deleted the interpolation-decay-api branch April 20, 2021 19:09
@elalish elalish mentioned this pull request May 11, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make camera controls physics configurable
2 participants