CGAL users discussion list

[Dominik Szczerba <>]

Hi,

Has the official version fixing this bug already been released?

Thanks
Dominik

2014-12-08 17:47 GMT+01:00 Jane Tournois 
<>:
> Dear Bryn,
>
> we have fixed the bug you pointed out. The fix will be available in the next
> public release of CGAL.
>
> Thanks again.
> Best regards,
>
> Jane.
>
> Le 20/10/2014 08:57, Bryn Lloyd a écrit :
>>
>>
>> Dear Jane
>>
>> Thanks, Bryn
>>
>>
>>
>> On 10/17/2014 8:55 AM, Jane Tournois wrote:
>>>
>>> Dear Bryn,
>>>
>>> thanks for the patch you suggested.
>>>
>>> I will take care of this and keep you informed of its integration to
>>> CGAL.
>>>
>>> Best regards,
>>> Jane.
>>>
>>> Le 14/10/2014 09:38, Bryn Lloyd a écrit :
>>>>
>>>> Dear CGAL developers
>>>>
>>>> I would like to submit a patch for a problem which occurs when
>>>> generating a 3D mesh from very large images. The detailed information is
>>>> below:
>>>>
>>>> OS, compiler:
>>>> - Windows 7 & 8, Visual Studio 2012
>>>>
>>>> Build options:
>>>> - Configured using cmake for VS 2012, without GMP, without ZLIB
>>>>
>>>> Description of the problem:
>>>> - Generating meshes from very large image (approx. 4e9 voxels) was not
>>>> working as expected. Sometimes one half of the image was meshed 
>>>> correctly,
>>>> but strange things were happening to the other half.
>>>>
>>>> Kernel (3D Mesh Generation from images/label fields):
>>>> - I was using an old version of CGAL (3.8), however, looking at the
>>>> newest version I can see that issue has not been fixed.
>>>>
>>>> Solution:
>>>> - After debugging a bit, I found that the function
>>>> 'Image_3::labellized_trilinear_interpolation' was using 'unsigned int'
>>>> instead of e.g. 'std::uint64_t' or size_t. This causes an integer 
>>>> overflow,
>>>> which breaks mesh generation.
>>>> Changing the code in Image_3.h and Image_to_labeled_function_wrapper.h
>>>> to use size_t fixed the issue.
>>>>
>>>> I attached a patch.
>>>>
>>>> Best regards
>>>> Bryn
>>>>
>>>>
>>>>
>>>>
>>>>
>>>
>>>
>>
>>
>
>
> --
> Jane Tournois, PhD
> R&D Engineer at GeometryFactory
> http://www.geometryfactory.com/
>
>
> --
> You are currently subscribed to cgal-discuss.
> To unsubscribe or access the archives, go to
> https://sympa.inria.fr/sympa/info/cgal-discuss
>
>