Dealga McArdle

this section of code is quite dense with features and side effects (meaning: certain lines of code have effect on things which are not explicitly mentioned, no mysteries just potential...

The issue of an empty per molecule can be dealt with separately.

weird bug. If the last 4 characters of the pdb name (not counting the extension) are all alphabetic, then this issue doesn't seem to happen. In the case where the...

I get the distinct sense that whoever coded `core_createModels():` had a bit of a battle with `BBModelRemark` ``` # these right sides a = copy.copy(str(bpy.context.scene.BBModelRemark)) b = str(bpy.context.scene.BBModelRemark) c =...

**copy.copy(x)** Using `a = copy.copy(x)` means the value of x is anticipated to change, but `a` is intended to carry the value that was present at the time the copy.copy...

There's a neater way to add an Empty. ``` python D = bpy.data scn = bpy.context.scene empty = D.objects.new('whatevername', None) # empty.location = vert_coordinate # empty.empty_draw_size = 0.45 scn.objects.link(empty) scn.update()...

I've added a single line fix for now, but in the EmptyTest branch. (it's possible to branch from a branch) The line that causes the rename is annotated here ```...

@faerietree I will wait for a while for the original author(s) to comment, they should be more intimately aware of the minefield involved in using these side effects. testing what...

in the meantime i've implemented it so it bypasses the issue, and would appreciate if you could test

changed to : https://github.com/MonZop/BioBlender/tree/alt_manual_dup_and_mlp