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

Joachim Pouderoux joachim.pouderoux at kitware.com
Wed Nov 25 05:17:52 EST 2015


We found the bug :) a strcmp != 0 was replaced by a std::string ==
std::string!!! Ben will be chasten for this! ;)


2015-11-25 10:24 GMT+01:00 Mathieu Westphal <mathieu.westphal at kitware.com>:

> 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
>>>
>>
>>
>
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Search the list archives at:
> http://markmail.org/search/?q=Paraview-developers
>
> Follow this link to subscribe/unsubscribe:
> http://public.kitware.com/mailman/listinfo/paraview-developers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/paraview-developers/attachments/20151125/a0944a64/attachment.html>


More information about the Paraview-developers mailing list