Skip to content

Animated view transforms R and T (Revisited)#56

Closed
nigels-com wants to merge 6 commits into
c42f:masterfrom
nigels-com:axis_alignment
Closed

Animated view transforms R and T (Revisited)#56
nigels-com wants to merge 6 commits into
c42f:masterfrom
nigels-com:axis_alignment

Conversation

@nigels-com

Copy link
Copy Markdown
Collaborator

Couldn't resist tinkering with #54

Rebased onto master branch and modestly refactored to be frame-rate independent.

c42f and others added 2 commits June 16, 2015 00:49
* Increase frame rate in an attempt to keep things smooth and
  non nausea-inducing for larger data sets.  (Or should we do this only
  during an animation?)

* Improve interpolation curves to use slerp() for angles and exponential
  scaling for radial distance.

* Ensure quaternions passed to InteractiveCamera are always normalized
@nigels-com

Copy link
Copy Markdown
Collaborator Author

Seems to me we don't need two separate timers. This change re-purposes the existing one to trigger redraws, and moves the animation logic into paintGL. The idea is for paintGL be called continuously for the duration of the animation.

@nigels-com

Copy link
Copy Markdown
Collaborator Author

Withdrawing this pull request, for now. (My branch now seems buggy)

@nigels-com nigels-com closed this Jun 16, 2015
@c42f

c42f commented Jun 16, 2015

Copy link
Copy Markdown
Owner

Ok, no probs. I think keeping the discussion in one place on #54 makes sense anyway. @gphysics has your first commit on his branch now.

@nigels-com

Copy link
Copy Markdown
Collaborator Author

Sure thiing.

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.

3 participants