Hi Robert.

Code was submit 06 April 2006. Some improvement was done since original
code, but global behaviors never change.
So issue was there since the beginning.

David

2015-02-03 20:55 GMT+01:00 Robert Osfield <[email protected]>:

> Hi David,
>
> It must be too late in the day for me to piece together the combinations
> you are trying to convey, it's just confusing me right now.  I haven't
> looked at the code or have data to test the problem.  At a random guess,
> could the issue be associated with the changes to osg::Geode so it's now an
> osg::Group.
>
> Robert.
>
> On 3 February 2015 at 13:00, David Callu <[email protected]> wrote:
>
>> Hi Robert,
>>
>> it seem that osgUtil::MergeGeodesVisitor only merge duplicate geode in a
>> same osg::Group.
>> But duplicate geode in a same osg::Group look like a bug rather than a
>> feature.
>>
>> More over,
>> if the same Geode is 2 time in the same osg::Group, the merged geode will
>> have 2 time the same drawable :
>> example :
>>   Before merge :
>>   Group A have 2 time the geode B as child, that have 2 drawable C and D.
>>   After merge
>>   Group A will have only one time Geode B that will have 2 time drawable
>> C and 2 time drawable D
>>
>> if the same Geode is 3 time in the same osg::Group, the merged geode will
>> have not 3 but 4 time the same drawable :
>> example :
>>   Before merge :
>>   Group A have 3 time the geode B as child, that have 2 drawable C and D.
>>   After merge
>>   Group A will have only one time Geode B that will have 4 time drawable
>> C and 4 time drawable D
>>
>>
>> What I expect from a MergeGeodeVisitor is to merge Geodes in a same
>> osg::Group in one Geode.
>> If a Geode are many time in the same group, I expect a BIG WARNING in the
>> log.
>>
>> If you agree with the concept, I will fix the MergeGeodeVisitor.
>>
>>
>>
>> PS:
>> this code seem to be copied from MergeGeometryVisitor that merge the
>> duplicate geometry only too,
>> with the same drawback that MergeGeodeVisitor.
>>
>>
>> David Callu
>>
>>
>> _______________________________________________
>> osg-submissions mailing list
>> [email protected]
>>
>> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>>
>>
>
> _______________________________________________
> osg-submissions mailing list
> [email protected]
>
> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
>
>
_______________________________________________
osg-submissions mailing list
[email protected]
http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org

Reply via email to