next »

[Marvin Fröhlich]

hi

I've just had an idea, how to improve performance for Models. We could (by setting a flag) ignore the "inner" bounds of the model. So no bounds of inner groups or shapes are computed, updated or even existing. Then the shapes would get unpickable of course. But a Group could get an additional flag called setPickUnit(boolean)/isPickUnit(). Then it would be considered to be "one" atomic node by the FrustumCuller and the PickingLibrary. This would even improve picking performance, since the PL would not need to dig into the Model's internals. And the Model itself would be returned as the picked Node.

What do you think? Any problems expected? I don't think so.

Marvin

[Patheros]

Sounds like a good idea. What if you have a model that has multiple distinct packable sections, but each section is composed of multiple shapes that you don't care about?

['n ddrylliog]

+1
To be tested.

[Marvin Fröhlich]

Sounds like a good idea. What if you have a model that has multiple distinct packable sections, but each section is composed of multiple shapes that you don't care about?

What exactly do you mean by "packable sections"? Something like that?:

Code:

--------------------
|****              |
|****              |
|******************|
|    **************|
|           *******|
|           *****  |
|           *****  |
|     ********     |
|     ****         |
|     ****         |
--------------------

Marvin

[horati]

What exactly do you mean by "packable sections"? Something like that?:

--------------------
|****              |
|****              |
|******************|
|    **************|
|           *******|
|           *****  |
|           *****  |
|     ********     |
|     ****         |
|     ****         |
--------------------

I don't want to put words in his mouth, but I think he meant to say "pickable sections".   Since we are loading from a model, imagine a spaceship with arms holding objects of interest.  We don't want to allow the player to pick the arms, just the interesting bits.  So the visible scene might look like:

--------------------
|****          ****|
|****          ****|
|******************|
|    **************|
|           *******|
|           *****  |
|           *****  |
|     ********     |
|     ****         |
|     ****         |
--------------------

but the pickable scene would only be

--------------------
|****          ****|
|****          ****|
|                  |
|                  |
|                  |
|                  |
|                  |
|                  |
|                  |
|                  |
--------------------

[Patheros]

What exactly do you mean by "packable sections"? Something like that?:

Wow, lots of talk about one word. It was meant to be be general. But if you want a more specific example think of a model with the following shapes in it: A B C D E F G H I J

And you want A C J to be one pickable unit, B D G I to be another, and E F H to be not pickable.

['n ddrylliog]

Easey

Code:

Model myModel = Loader.load();
Group acj = new Group();
acj.addChild(myModel.getNode("A"));
acj.addChild(myModel.getNode("C"));
acj.addChild(myModel.getNode("J"));
acj.setPacked(true);
Group bdgi = new Group();
bdgi.addChild(myModel.getNode("B"));
bdgi.addChild(myModel.getNode("D"));
bdgi.addChild(myModel.getNode("G"));
bdgi.addChild(myModel.getNode("I"));
acj.setPacked(true);
Group efh = new Group();
efh.addChild(myModel.getNode("E"));
efh.addChild(myModel.getNode("F"));
efh.addChild(myModel.getNode("H"));
efh.setPacked(true);
efh.setPickable(false);
scene.addChild(acj);
scene.addChild(bdgi);
scene.addChild(efh);

The setPacked() method is missing

[Marvin Fröhlich]

Wouldn't this hurt the unique structure of the scenegraph?

But the above request could be solved by not setting the Model itself as a pickunit, but certain subgroups of it. And remember, the pickunit feature will always be optional.

Marvin

[Patheros]

In Amos' code I would assume that the actual Model is never added to the scenegraph.

I was really just thinking about different possibilities. I'm not sure if theres is an actual use for it. 

[horati]

Now, I understand what you're going for.  In this instance, I think you need to:

• load the model NOT pickable
• calculate the bounding sphere that encompasses A C J
• calculate the bounding sphere that encompasses B D G I
• set the model pickable without recursion; i.e., setPickable( true, false )
• add 2 invisible objects, make them pickable, and set their bounding spheres to the values calculated above

Marvin, do you have any ideas for optimizing this?

[Marvin Fröhlich]

Marvin, do you have any ideas for optimizing this?

In principle it is exactly, what I intended to say with the pickunit idea . So this will be one of my next tasks.

Marvin