[Paraview] ANN: Upcoming changes to ParaView Server Manager

[Sebastien Jourdain]

Hi Folks,

I've push some new code in the same stage branch that should fix the
plugins (coprocessing, manta, EyeDomLighting...) and some other minor
bugs.

Sven also provide me some feedback that may be interesting to share:
For those with custom readers that internally use FileSeriesReader:

* Replace in servermanager XML:

    <FileSeriesReaderProxy name="..." ...>

 with

    <SourceProxy name="..." ... si_class="vtkSIFileSeriesReaderProxy">

Moreover, you can look into the git history and see what I did change
to make those plugin work.
The manta one could be interesting for some of you. (git show 2aa56d)

Enjoy,

Seb


2011/3/29 Takuya OSHIMA <oshima at eng.niigata-u.ac.jp>:
> Thanks again for the clarification!
>
> Takuya
>
> Takuya OSHIMA, Ph.D.
> Faculty of Engineering, Niigata University
> 8050 Ikarashi-Ninocho, Nishi-ku, Niigata, 950-2181, JAPAN
>
> From: Sebastien Jourdain <sebastien.jourdain at kitware.com>
> Subject: Re: [Paraview] ANN: Upcoming changes to ParaView Server Manager
> Date: Tue, 29 Mar 2011 09:34:09 -0400
>
>> Hi Takuya,
>>
>> >> NEW2: pxy->ExecuteStream(stream)
>> >
>> > In fact, this is not possible because vtkSMProxy::ExecuteStream() is
>> > protected (is this a bug or intended?).
>>
>> It is intended, specially if your are not inside the vtkSMProxy class
>> implementation.
>> It is more a helper method that use the proxy session and location to
>> send and execute the stream that may not be related in any other way
>> to that proxy.
>>
>> thanks,
>>
>> Seb
>>
>> >
>> > Takuya
>> >
>> > Takuya OSHIMA, Ph.D.
>> > Faculty of Engineering, Niigata University
>> > 8050 Ikarashi-Ninocho, Nishi-ku, Niigata, 950-2181, JAPAN
>> >
>> >
>> > From: Sebastien Jourdain <sebastien.jourdain at kitware.com>
>> > Subject: Re: [Paraview] ANN: Upcoming changes to ParaView Server Manager
>> > Date: Tue, 29 Mar 2011 08:37:14 -0400
>> >
>> >> Hi Takuya,
>> >>
>> >> I'm glad that you gave a try and you managed to make your plugin work.
>> >> Moreover, thanks for that "cheat sheet".
>> >>
>> >> Here is my feedback about it:
>> >>
>> >>> OLD: pxy->Set/GetServers() (where pxy is a vtkSMProxy)
>> >>> NEW: pxy->Set/GetLocation()
>> >>
>> >> Yes
>> >>
>> >>> OLD: pxy->Set/GetConnectionID()
>> >>> NEW: pxy->Set/GetSession()
>> >>
>> >> Yes, but those lines should simply be removed as the session is set on
>> >> the Proxy at its creation, therefore there is no point in doing it
>> >> twice.
>> >>
>> >>> OLD: pxy->GetID()
>> >>> NEW: VTKOBJECT(pxy) (a stream operator in client-server stream)
>> >>
>> >> Yes
>> >>
>> >>> OLD: pm->SendSteram(this->GetConnectionID(), this->GetServers(), stream)
>> >>> NEW: this->ExecuteStream(stream) (where this is a vtkSMProxy)
>> >>
>> >> Yes
>> >>
>> >>> OLD: pm->SendStream(pxy->GetConnectionID(), pxy->GetServers(), stream)
>> >>> NEW: pxy->GetSession()->ExecuteStream(pxy->GetLocation(), stream)
>> >>> (where pm is a vtkProcessModule)
>> >>
>> >> Yes, but as you did it previously you can also do that
>> >> NEW2: pxy->ExecuteStream(stream)
>> >>
>> >>> Also, what I noticed so far are
>> >>> * CoProcessing module does not build due to some missing headers.
>> >>
>> >> I'm going to look into that and fix it. Thanks to have notice it.
>> >>
>> >>> * ParaViewCore/ServerManager/vtkSMDataSourceProxy.{cxx,h} are orphaned
>> >>>  (have no reference from other sources or CMakeLists).
>> >>
>> >> Yes, I was planning to remove it. Are you using it ?
>> >>
>> >> Thanks again for those valuable comments,
>> >>
>> >> Seb
>> >
>