Molecule orientation not applied about cellblender

[GoogleCodeExporter]

The following line (1612 as of this writing) in cellblender_operators.py is no 
longer working:

mol_pos_mesh.vertices.foreach_set("normal", mol_orient)

However it is still possible to set the normals manually as in the following 
for loop:

for idx, vert in enumerate(mol_pos_mesh.vertices):
    setattr(vert, "normal", mathutils.Vector((mol_orient[idx*3], mol_orient[idx*3+1], mol_orient[idx*3+2])))

I have not timed this to see if it's any slower than the original method. This 
would be worth looking into.

Also, I posted a bug report to the Blender developers about this problem, and 
it's targeted to be investigated.

Original comment by [email protected] on 27 Feb 2013 at 7:49

[GoogleCodeExporter]

On second thought, I suppose this would be a more sensible/clearer alternative 
than using setattr:

for idx, vert in enumerate(mol_pos_mesh.vertices):
    vert.normal = mathutils.Vector((mol_orient[idx*3], mol_orient[idx*3+1], mol_orient[idx*3+2]))

:)

Original comment by [email protected] on 27 Feb 2013 at 8:10

[GoogleCodeExporter]

It's been reported that there is a fix in the Blender svn repo.  

Original comment by [email protected] on 28 Feb 2013 at 5:04

[GoogleCodeExporter]

The workaround is definitely slower. With 100,000 molecules, it takes about 
0.004 seconds to set the normals with the original method and about 0.135 
seconds with the workaround.

Original comment by [email protected] on 28 Feb 2013 at 8:23

[GoogleCodeExporter]

A fix was added for Blender 2.66a.

Original comment by [email protected] on 12 Mar 2013 at 4:15

• Changed state: Fixed