[Paraview] Bug in Paraview 4.2 example + fix

[B.W.H. van Beest]

Hi Cory,

Thanks for putting the fix in. Though willing to contribute, I have no
experience with gerrit.  So I don't know what to do, really.

Regards,
Bertwim


On 10/10/2014 08:47 PM, Cory Quammen wrote:
> Hi Bertwim,
>
> Thanks for reporting this bug and proposing a fix.
>
> I posted your report as a bug in the bug tracker:
>
> http://www.paraview.org/Bug/view.php?id=15044
>
> I also pushed a git topic to gerrit with your fix and some fixes for
> the HiddenLinesRemoval plugin. Please feel free to review it:
>
> http://review.source.kitware.com/#/t/4837/
>
> Thanks,
> Cory
>
> On Fri, Oct 10, 2014 at 1:53 PM, B.W.H. van Beest <bwvb at xs4all.nl> wrote:
>> Could perhaps somebody of Kitware react to this posting of a bug?
>> I noticed a similar issue with the Example plugin "HiddenLinesRemoval",
>> although in that case there seems to be more wrong.
>>
>> Regards,
>> Bertwim
>>
>>
>> On 10/07/2014 10:30 PM, B.W.H. van Beest wrote:
>>> Hello,
>>>
>>> I think there is a bug in the plugin example in
>>> ../ParaView/Examples/Plugin/Representation (Paraview 4.2)
>>>
>>> As given, loading the plugin manually, and then creating a new source
>>> (e.g. Box) causes Paraview to crash with the error message:
>>>
>>>> ERROR: In
>>>> /usr/local/git/ParaView/ParaViewCore/ServerImplementation/Core/vtkSIProxyDefinitionManager.cxx,
>>>> line 565
>>>> vtkSIProxyDefinitionManager (0x265ebb0): No proxy that matches:
>>>> group=representations and proxy=SurfaceRepresentationBase were found.
>>>>
>>>>
>>>> ERROR: In
>>>> /usr/local/git/ParaView/ParaViewCore/ServerImplementation/Core/vtkSIProxyDefinitionManager.cxx,
>>>> line 989
>>>> vtkSIProxyDefinitionManager (0x265ebb0): Failed to locate base proxy
>>>> definition (representations, SurfaceRepresentationBase). Aborting for
>>>> debugging purposes.
>>>>
>>>>
>>>> Abort
>>> The error seems to be in the file Representation.xml, where the attribute
>>> base_proxyname is set to "SurfaceRepresentationBase", which should be
>>> "SurfaceRepresentation" according to the documentation in
>>> http://www.paraview.org/Wiki/ParaView/Plugin_HowTo#Adding_new_Representations_for_3D_View_using_Plugins_.2A_new_in_version_3.7
>>>
>>> When I make the change, recompile the plugin and load in into Paraview,
>>> there is no longer the above crash when I create a new Box, and the
>>> representation "Special Mapper" feautures in the Representationnn combobox.
>>>
>>>> [243] Representation> diff Representation.xml.org Representation.xml
>>>> 7c7
>>>> <       base_proxyname="SurfaceRepresentationBase">
>>>> ---
>>>>>       base_proxyname="SurfaceRepresentation">
>>> Kind Regards,
>>> Bertwim
>>>
>>> _______________________________________________
>>> Powered by www.kitware.com
>>>
>>> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>>>
>>> Please keep messages on-topic and check the ParaView Wiki at: http://paraview.org/Wiki/ParaView
>>>
>>> Follow this link to subscribe/unsubscribe:
>>> http://public.kitware.com/mailman/listinfo/paraview
>>>
>>>
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>>
>> Please keep messages on-topic and check the ParaView Wiki at: http://paraview.org/Wiki/ParaView
>>
>> Follow this link to subscribe/unsubscribe:
>> http://public.kitware.com/mailman/listinfo/paraview
>