[Paraview-developers] [vtk-developers] Updated VTK Breaks ParaView tests

[Mathieu Westphal]

Hi Ben

I'e bissecting the commit and found the cullprint, looks like this one
breaks it :
https://gitlab.kitware.com/vtk/vtk/commit/68c3cc53e47c30e7ef8a74fa2b40ced0f234834c


Mathieu Westphal

On Tue, Nov 24, 2015 at 7:13 PM, David Gobbi <david.gobbi at gmail.com> wrote:

> Hi Mathieu,
>
> The errors look similar to ones that were fixed by this commit:
> https://gitlab.kitware.com/paraview/paraview/commit/8a6e6fe9
>
> At the very root, bugs such as the ones fixed by the above commit
> are caused by type checks of the form "type{x) == y".   Such type
> checks should almost always be replaced by calls to "isinstance()"
> or "issubclass()".
>
> I did a quick grep through the ParaView source code to find places
> where type() is being used to do type checks, and I found several.
> I'm pretty sure that at least some of these are bugs:
>
> Wrapping/Python/paraview/coprocessing.py:54:        if type(frequencies)
> != dict:
> Wrapping/Python/paraview/data_exploration.py:204:                    if
> type(value) == type("String"):
> Wrapping/Python/paraview/extract_selection.py:88:            (query,
> type(maskArray))
> Wrapping/Python/paraview/servermanager.py:654:        if not type(self) is
> Property:
> Wrapping/Python/paraview/servermanager.py:671:        if type(self) is
> Property:
> Wrapping/Python/paraview/servermanager.py:888:        if type(value) ==
> str:
> Wrapping/Python/paraview/servermanager.py:2295:    elif type(arg1) is
> types.IntType:
> Wrapping/Python/paraview/servermanager.py:3038:    if not type(val) == int:
> Wrapping/Python/paraview/simple.py:740:    if type(filename) == list:
> Wrapping/Python/paraview/smtrace.py:397:            if not type(prop) ==
> sm.Property:
> Wrapping/Python/paraview/smtrace.py:490:        assert type(propertyname)
> == str
> Wrapping/Python/paraview/smtrace.py:1112:            return "'%s'" % x if
> type(x) == str else x
> Web/Applications/FileViewer/server/pv_web_file_loader.py:136:        if
> type(files) == list:
> Web/Python/paraview/web/helper.py:249:            if type(data) in
> allowedTypes:
> Web/Python/paraview/web/helper.py:272:            elif type(prop) ==
> ProxyProperty:
> Web/Python/paraview/web/helper.py:297:             if
> property.GetDomain('proxy_list') and len(value) == 1 and type(value[0]) ==
> str:
> Web/Python/paraview/web/helper.py:315:    if type(value) == unicode:
> Web/Python/paraview/web/helper.py:317:    if type(value) == list:
> Web/Python/paraview/web/helper.py:349:       if
> type(proxy.GetProperty(property)) == ProxyProperty:
> Web/Python/paraview/web/protocols.py:1192:                if type(prop) ==
> ProxyProperty or type(prop) == InputProperty:
> Web/Python/paraview/web/protocols.py:1590:        if type(relativePath) ==
> list:
> Web/Python/paraview/web/protocols.py:1933:        if type(relativePath) ==
> list:
> Web/Python/paraview/web/protocols.py:1962:                if
> type(lut['name']) == unicode:
> Web/Python/paraview/web/pv_web_catalyst.py:202:        if type(files) ==
> list:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:54:    elif
> type(array) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:74:    if
> type(array1) == dsa.VTKCompositeDataArray and type(val2) ==
> dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:83:    elif
> type(array1) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:112:        if
> type(array) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:132:        if
> type(ds) == dsa.CompositeDataSet:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:176:    if
> type(array) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:341:    t =
> type(array)
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:700:        if
> type(array) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:727:        if
> type(array) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:753:    if
> type(array) == dsa.VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/algorithms.py:776:    if
> type(arrayx) == dsa.VTKCompositeDataArray and type(arrayy) ==
> dsa.VTKCompositeDataArray and (type(arrayz) == dsa.VTKCompositeDataArray or
> arrayz is None):
> VTK/Wrapping/Python/vtk/numpy_interface/dataset_adapter.py:515:        if
> type(index) == VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/dataset_adapter.py:534:        if
> type(other) == VTKCompositeDataArray:
> VTK/Wrapping/Python/vtk/numpy_interface/dataset_adapter.py:555:        if
> type(other) == VTKCompositeDataArray:
>
>  - David
>
>
>
>
>
> On Tue, Nov 24, 2015 at 4:04 AM, Mathieu Westphal <
> mathieu.westphal at kitware.com> wrote:
>
>> Hello
>>
>> I am trying to update VTK in ParaView in order to integrate my GIL
>> ensured changes, but a lot of python Tests are failing with paraview, with
>> the following error :
>>
>> Traceback (most recent call last): File
>> "/home/kitware/Dashboards/buildslave/paraview-amber8-linux-static-release_mpi_osmesa_python/source/ParaViewCore/ServerManager/Default/Testing/Python/MultiView.py",
>> line 9, in <module> Show() File
>> "/home/kitware/Dashboards/buildslave/paraview-amber8-linux-static-release_mpi_osmesa_python/build/lib/site-packages/paraview/simple.py",
>> line 400, in Show rep = controller.Show(proxy, proxy.Port, view) File
>> "/home/kitware/Dashboards/buildslave/paraview-amber8-linux-static-release_mpi_osmesa_python/build/lib/site-packages/paraview/servermanager.py",
>> line 158, in __getattr__ return getattr(self.SMController, name)
>> AttributeError: 'vtkPVServerManagerCorePython.vtkSMParaViewPipeline' object
>> has no attribute 'Show' Traceback (most recent call last): File
>> "/home/kitware/Dashboards/buildslave/paraview-amber8-linux-static-release_mpi_osmesa_python/source/ParaViewCore/ServerManager/Default/Testing/Python/MultiView.py",
>> line 9, in <module> Show() File
>> "/home/kitware/Dashboards/buildslave/paraview-amber8-linux-static-release_mpi_osmesa_python/build/lib/site-packages/paraview/simple.py",
>> line 400, in Show rep = controller.Show(proxy, proxy.Port, view) File
>> "/home/kitware/Dashboards/buildslave/paraview-amber8-linux-static-release_mpi_osmesa_python/build/lib/site-packages/paraview/servermanager.py",
>> line 158, in __getattr__ return getattr(self.SMController, name)
>> AttributeError: 'vtkPVServerManagerCorePython.vtkSMParaViewPipeline' object
>> has no attribute 'Show'
>>
>>
>>
>> https://open.cdash.org/index.php?compare1=63&filtercount=2&field1=buildname%2Fstring&project=ParaView&field2=buildstarttime%2Fdate&showfilters=0&limit=100&compare2=83&value1=dbdbee7a&showfeed=0&value2=20151124T043202
>>
>> https://gitlab.kitware.com/paraview/paraview/merge_requests/329#note_47210
>>
>>
>> I Suspect it is because of a VTK commit, and i will try to recreate it
>> locally by only updating VTK and bisecting the commits.
>>
>> But if anyone has an idea about it, that would be nice.
>>
>> Mathieu Westphal
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/paraview-developers/attachments/20151125/4c4672cd/attachment-0001.html>