Incorrect RNA Path when trying to keyframe the nth vertex of Bezier curve shape key data #38013

New Issue

Joshua Leung · 2014-01-01T12:10:00+01:00

Joshua Leung commented

2014-01-01 12:10:00 +01:00

System Information
Windows 64-bit, OpenSuse (64-bit?)

Blender Version
Broken: ead6d397fd
Worked: (optional)

Short description of error
Incorrect RNA Paths are generated for shape key data elements on bezier curves when trying to keyframe these.

Exact steps for others to reproduce the error
See the following file from @zanqdo - Curve-ShapekeyDataPathBug.blend
Simply press enter in the python console

**System Information** Windows 64-bit, OpenSuse (64-bit?) **Blender Version** Broken: ead6d397fd Worked: (optional) **Short description of error** Incorrect RNA Paths are generated for shape key data elements on bezier curves when trying to keyframe these. **Exact steps for others to reproduce the error** See the following file from @zanqdo - [Curve-ShapekeyDataPathBug.blend](https://archive.blender.org/developer/F60077/Curve-ShapekeyDataPathBug.blend) Simply press enter in the python console

Joshua Leung commented

2014-01-01 12:10:00 +01:00

Changed status to: 'Open'

Joshua Leung self-assigned this 2014-01-01 12:10:00 +01:00

Joshua Leung commented

2014-01-01 12:10:00 +01:00

Added subscriber: @JoshuaLeung

Joshua Leung commented

2014-01-01 12:11:53 +01:00

Added subscriber: @zanqdo

Joshua Leung commented

2014-01-01 12:22:55 +01:00

NOTE: I'm just submitting this as a bug report here so that the reference test files for this are easily available online, should further problems with the fix for this that I've been working on.

Here's a second reference file Curve-ShapekeyDataPathBug-02.blend , containing several additional object types (each one has two shapekeys), and additional vertices added to the test curve (which helped pick up a bug in earlier fixes).

The Problem
key->elemsize is the wrong size for Bezier Curves. By default, it has a value of 16, which only works when dealing with nurbs curves/surfaces but not bezier curves. In various places, quite a bit of evil magic is performed to adjust elemsize and count of number of elements to work around the problems.

The Current Fix
Notably, for whatever reason, the "magic number" of 12 seems to be getting applied for the number of floats stored per bezier point in the shapekey, just like for the other data types (e.g. mesh and lattice). For now, when dealing with ShapeKeyBezierPoints instead of ShapeKeyPoints (i.e. the special type of point used for shape key data on Bezier curves), we use this special value instead.

NOTE: I'm just submitting this as a bug report here so that the reference test files for this are easily available online, should further problems with the fix for this that I've been working on. Here's a second reference file [Curve-ShapekeyDataPathBug-02.blend](https://archive.blender.org/developer/F60078/Curve-ShapekeyDataPathBug-02.blend) , containing several additional object types (each one has two shapekeys), and additional vertices added to the test curve (which helped pick up a bug in earlier fixes). **The Problem** key->elemsize is the wrong size for Bezier Curves. By default, it has a value of 16, which only works when dealing with nurbs curves/surfaces but not bezier curves. In various places, quite a bit of evil magic is performed to adjust elemsize and count of number of elements to work around the problems. **The Current Fix** Notably, for whatever reason, the "magic number" of 12 seems to be getting applied for the number of floats stored per bezier point in the shapekey, just like for the other data types (e.g. mesh and lattice). For now, when dealing with ShapeKeyBezierPoints instead of ShapeKeyPoints (i.e. the special type of point used for shape key data on Bezier curves), we use this special value instead.

blender-admin commented

2014-01-01 12:28:39 +01:00

This issue was referenced by blender/blender-addons-contrib@c8dd6b67b6

This issue was referenced by blender/blender-addons-contrib@c8dd6b67b6de00d4839d76d0cbffe3d187995cdc

blender-admin commented

2014-01-01 12:28:39 +01:00

This issue was referenced by c8dd6b67b6

This issue was referenced by c8dd6b67b6de00d4839d76d0cbffe3d187995cdc

Joshua Leung commented

2014-01-01 12:33:10 +01:00

Changed status from 'Open' to: 'Resolved'

Joshua Leung closed this issue