[Bf-blender-cvs] CVS commit: blender/source/blender/include BIF_editaction.h BSE_editipo.h blender/source/blender/makesdna DNA_scene_types.h blender/source/blender/src editaction.c editipo.c header_action.c header_ipo.c space.c

[Joshua Leung]

aligorith (Joshua Leung) 2006/11/09 09:43:27 CET

  Modified files:
    blender/source/blender/include BIF_editaction.h BSE_editipo.h 
    blender/source/blender/makesdna DNA_scene_types.h 
    blender/source/blender/src editaction.c editipo.c 
                               header_action.c header_ipo.c 
                               space.c 
  
  Log:
  This commit adds two of my recent animation editing related patches:
  #5061 - Ipo/Action 'Cleaning'
  #5071 - 'Only Needed' Keyframing Option
  
  ====================
  
  * IPO/Action 'Cleaning':
  It removes un-necessary keyframes from individual ipo curves.
  - In both editors, the hotkey is currently the OKEY.  Also accesable from menus of each editor.
  - There is currently a 'threshold' popup. This sets the value that the cleaner uses to determine if two keys have same time/value
  
  There are a few improvements that could still be made, such as:
  - There are a few cases that it still doesn't handle yet, such as when un-needed keyframes lie on a linear line (and similiar cases). This shall be improved soon.
  - Also, for some reason, after running cleaning while in ipo editor editmode, all but the active curve are hidden.
  
  ====================
  
  * 'Only Needed' Keyframing Option:
  This patch adds a new keyframing option for objects and bones. It only adds keyframes where they are needed, judging from the surrounding points on that curve.
  
  Notes about this keyframing option:
  - Works like the existing 'Avail' option, except it checks if the keyframe
  is needed.
  - Currently uses hardcoded threshold for determining if same value.
  
  [quote]
  /* Cases where keyframes should not be added:
  *       1. Keyframe to be added bewteen two keyframes with similar values
  *       2. Keyframe to be added between two keyframes with similar times
  *       3. Keyframe lies at point that intersects the linear line between two
  keyframes
  */
  [/unquote]
  
  Revision  Changes    Path
  1.13      +3 -1      blender/source/blender/include/BIF_editaction.h
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/include/BIF_editaction.h.diff?r1=1.12&r2=1.13&cvsroot=bf-blender>
  1.26      +5 -1      blender/source/blender/include/BSE_editipo.h
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/include/BSE_editipo.h.diff?r1=1.25&r2=1.26&cvsroot=bf-blender>
  1.81      +5 -1      blender/source/blender/makesdna/DNA_scene_types.h
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/makesdna/DNA_scene_types.h.diff?r1=1.80&r2=1.81&cvsroot=bf-blender>
  1.83      +90 -2     blender/source/blender/src/editaction.c
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/src/editaction.c.diff?r1=1.82&r2=1.83&cvsroot=bf-blender>
  1.107     +341 -4    blender/source/blender/src/editipo.c
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/src/editipo.c.diff?r1=1.106&r2=1.107&cvsroot=bf-blender>
  1.44      +13 -1     blender/source/blender/src/header_action.c
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/src/header_action.c.diff?r1=1.43&r2=1.44&cvsroot=bf-blender>
  1.53      +8 -1      blender/source/blender/src/header_ipo.c
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/src/header_ipo.c.diff?r1=1.52&r2=1.53&cvsroot=bf-blender>
  1.394     +4 -1      blender/source/blender/src/space.c
    <http://projects.blender.org/viewcvs/viewcvs.cgi/blender/source/blender/src/space.c.diff?r1=1.393&r2=1.394&cvsroot=bf-blender>