[vtk-developers] vtkVoxelModeller converted from BIT to INT

[Will Schroeder]

Hi David-

I think we have to approach this in a series of steps, we may not be able to
correctly solve the VTK_BIT problem immediately, but do something quick with
an eye to fixing it longer term.

One option: since the writer doesn't work now let's do something quickly,
documenting the fact (e.g. bug tracker, etc.) that long term we need to
improve the support for VTK_BIT. (I would use VTK_UNSIGNED_CHAR myself which
is half the size of VTK_SHORT assuming it works.)

Another option (and I am shooting from the hip here) is to do the minimal
work in the reader/writer to get VTK_BIT "sort of" working meaning you may
have to perform temporary conversions to and from VTK_BIT to another type,
etc. which may cause temporary increases in memory.

I'm sure there are other opinions in the community about this...

Will

On Mon, Mar 15, 2010 at 11:01 AM, David Doria
<daviddoria+vtk at gmail.com<daviddoria%2Bvtk at gmail.com>
> wrote:

> On Mon, Mar 15, 2010 at 10:35 AM, Bill Lorensen <bill.lorensen at gmail.com>wrote:
>
>> I didn't say phased out, I said not well supported. Here isw what I
>> said last fall:
>>
>> "I think this is a bug in the writer. The scalar data type is VTK_BIT.
>> I'm, guessing the writer can't handle this type.
>>
>> I changed the scalar type to short in my local copy of vtk and the
>> program ran without warnings. However, I still could not read the file
>> in paraview. But I changed the writer to a vtkDataSetWriter and it ran
>> without warning. In this case, when the type was short, I was able to
>> see the voxels. As VTK_BIT I could not.
>>
>> VTK_BIT is not a well-supported type."
>>
>
> Will,
>
> Would VTK_SHORT still use too much memory? It just seems much much simpler
> to change 2 lines in vtkVoxelModeller - a change to a writer seems like it
> would never get approved.
>
> Thanks,
>
> David
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20100315/8d5cdd66/attachment.html>