How to add entities to a nested group?

draftomatic

@tig said:

The 'groupGroup' group DOESN'T really exist... except that you have made an empty group [too early] with that reference.
Adding that empty group so early on into the mix and then trying to 'move' it into another group only confuses things.
IF you already have a non-empty group with that reference then it's OK... but if not only make it when you need it otherwise.

In my real code, there is already a CPoint inside "groupGroup." If you think it's a problem that groupGroup is empty, then fine, I just tried adding a CPoint to groupGroup in my example script. Still broken.

@tig said:

If you still can't get your head around this

Sir, my head is wrapped around this problem about 9 times already. I feel like you think I don't understand, but I do. I understand every word you have written, and I keep telling you, "I'm doing that, I'm doing that. Look at my code, look at my code."

@tig said:

please post a simple before and after set of uniquely named objects with their hierarchy - e.g.

I did this already. Look at my script!!! Look at my screenshots!!! I am creating rootGroup and groupGroup. That is my hierarchy.

I'm going to repeat my question again. What is wrong with line 49 of my previous example script? I COMPLETELY understand EVERYTHING you're telling me, and I believe I'm doing it, but it's still broken, and I don't see why!!!!!!!!!!

TIG

Aaargh... in your sample code you 'magick' a group out of 'thin air' [groupGroup] and then you add it into a group within a nested group. IF that does exists it certainly isn't inside the 'newRootGroup' [retitled 'rootGroup'] entities as you've just created that - so it will ultimately splat - you ARE cross-threading entities.

Please try recasting your code [ignoring names and reusing references [variables] as it's getting confusing] in a way I outlined...

draftomatic

I think TIG gave up on me...

draftomatic

@tig said:

Aaargh...
Please try recasting your code [ignoring names and reusing references [variables] as it's getting confusing] in a way I outlined...

Argh indeed!

Here it is - as simple as I can make it. This is 14 lines of code. Now tell me, what is wrong?

The heirarchy is: model.entities -> group1 -> group2
After the code finishes, it should be: model.entities -> group1 -> group2 -> faceGroup -> face+edges

You can see that I'm creating these groups here, so there's no "magick!"

` # Testing group nesting

model = Sketchup.active_model
modelEntities = model.active_entities

Create a face

pts = []
pts[0] = [0, 0, 0]
pts[1] = [100, 0, 0]
pts[2] = [100, 100, 0]
pts[3] = [0, 100, 0]
face = modelEntities.add_face(pts)

Create root group

group1 = modelEntities.add_group()

Nest group2 inside group1

group2 = group1.entities.add_group()

Create group for faceGroup

faceGroup = modelEntities.add_group(face)

Nest in the first level

newGroup1 = modelEntities.add_group(group1, faceGroup)
group1.explode()

Repeat to get it to the second level deep

newGroup2 = newGroup1.entities.add_group(group2, faceGroup)

group2.explode() # This crashes SketchUp`

Chris Fullmer

You are exploding group1 though. That does seem right to me. I can't look at it right now, but I probably can try later tonight.

Seems like you should make a group and add the face data to it. Then make a group and add the facegroup to it. Then make a new group and add the other group to that. I'm sure that is exactly what you've already tried. I'll see if I can write up a fully working sample (unless someone else beats me to it).

Chris

draftomatic

Thank you very much Chris. And thanks to TIG too, even though I'm pretty sure he's moved on to bigger and better threads than this =D

draftomatic

Check it out. I have all these groups (group2 - groupN), under a root group (group1). And then I have faces, which I want to group with edges (faceGroup), and place in a nested group (group2 - groupN). There could potentially be many faceGroups, as shown in the picture...

HOWEVER... my example script 2 posts up is a simplified version of this, with only one nested group (group2), which is initially empty.

Clear enough?

su grouping bug storyboard.JPG

thomthom

I'm confused by all this...
Why are you not creating the entities directly into the appropriate group instead of trying to move them from group to group?

draftomatic

@thomthom said:

I'm confused by all this...
Why are you not creating the entities directly into the appropriate group instead of trying to move them from group to group?

Because, my "real" code gets the faces by way of the user clicking a context menu on an existing face and clicking my plugin's option. So, the face is already there, but my example script is emulating this by creating a face inside of model.entities first...

thomthom

ok...

What about:

user pick face
group the face ( group1)
get the definition of the group ( group1.entities.parent)
add a new instance of the group into the group you want it
erase group1

When you have the group instance you can use entities.add_instance to add a copy of the group.
It might avoid the problems if using .add_group with entities as argument and exploding.

draftomatic

@thomthom said:

ok...

What about:

user pick face

group the face ( group1)

get the definition of the group ( group1.entities.parent)

add a new instance of the group into the group you want it

erase group1

When you have the group instance you can use entities.add_instance to add a copy of the group.
It might avoid the problems if using .add_group with entities as argument and exploding.

Oh... tricky. I'll try that tomorrow.

thomthom


# picked_face is the face the user selected
# group2 is the destination group
group1 = picked_face.parent.entities.add_group( picked_face.all_connected )
group_def = group1.entities.parent
group2.entities.add_instance( group_def, group2.transformation.inverse * group1.transformation )
group1.erase!

draftomatic

YAY! Thanks a ton thomthom, your solution works.