<div dir="ltr"><div>Hi All, <br></div><div><br></div><div>Thanks for the feedback.</div><div><br></div><div>Based on feedback, we will keep the widget QVTKOpenGLWidget and only rename <span class="gmail-im">QVTKOpenGLSimpleWidget to </span>QVTKOpenGLNativeWidget<br></div><div>The corresponding change can be found at <a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4642">https://gitlab.kitware.com/vtk/vtk/merge_requests/4642</a></div><div><br></div><div><br></div><div>> unless it's somehow possible to make a widget that will work well in both scenarios</div><div><br></div><div>That is the idea, ultimately we would like the QVTKOpenGLWidget to be smarter.</div><div><br></div><div>Thanks</div><div>Jc<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 28, 2018 at 1:28 AM Elvis Stansvik <<a href="mailto:elvis.stansvik@orexplore.com">elvis.stansvik@orexplore.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">2018-08-28 0:12 GMT+02:00 Jean-Christophe Fillion-Robin <<a href="mailto:jcfr@kitware.com" target="_blank">jcfr@kitware.com</a>>:<br>
> Hi All,<br>
><br>
> To follow up on Mathieu email, we are in in the process of integrating this<br>
> merge request [1] (and the equivalent in paraview [2]) that will add/rename<br>
> the following classes:<br>
><br>
> * QVTKOpenGLAlienWidget  (equivalent to QVTKOpenGLWidget class described in<br>
> the previous email)<br>
><br>
> * QVTKOpenGLNativeWidget (equivalent to QVTKOpenGLSimpleWidget class<br>
> described in the previous email)<br>
><br>
> * QVTKOpenGLWidget will:<br>
>    - behave as it was before the introduction of the "new QVTKOpenGLWidget"<br>
> and will instead derive from QVTKOpenGLNativeWidget.<br>
>    - only be available if VTK_LEGACY_REMOVE is OFF<br>
>    - will most likely be removed in VTK 10<br>
><br>
><br>
> We anticipate that this clearer naming convention will facilitate update of<br>
> user code.<br>
><br>
> Let us know what you think,<br>
<br>
I think this naming is the best one possible, unless it's somehow<br>
possible to make a widget that will work well in both scenarios of<br>
course :)<br>
<br>
Much better naming than the "Simple" suffix, so +1.<br>
<br>
Elvis<br>
<br>
><br>
> Thanks<br>
> Jc<br>
><br>
><br>
> [1] <a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/4625" rel="noreferrer" target="_blank">https://gitlab.kitware.com/vtk/vtk/merge_requests/4625</a><br>
> [2] <a href="https://gitlab.kitware.com/paraview/paraview/merge_requests/2693" rel="noreferrer" target="_blank">https://gitlab.kitware.com/paraview/paraview/merge_requests/2693</a><br>
><br>
><br>
><br>
><br>
><br>
><br>
><br>
><br>
> On Thu, Aug 23, 2018 at 5:07 AM Mathieu Westphal<br>
> <<a href="mailto:mathieu.westphal@kitware.com" target="_blank">mathieu.westphal@kitware.com</a>> wrote:<br>
>><br>
>> Hello list,<br>
>><br>
>> If you are working on a VTK/Qt application this information should<br>
>> interest you.<br>
>> Sometime ago, a new QVTKOpenGLWidget implementation has been added to VTK,<br>
>> while the old one has been moved to QVTKOpenGLSimpleWidget.<br>
>> The last fixes for this change have just been merged, so make sure to use<br>
>> VTK master to test this.<br>
>><br>
>> 1. Why is there a new widget and what does it do ?<br>
>> We have been having some issues reported for the old widget, and the old<br>
>> widget could not support quad buffer stereo rendering by design.<br>
>> Has it is a needed feature in ParaView, a reimplementation was necessary.<br>
>> This new widget fix most of the reported issues with the old widget as well<br>
>> as adding stereo support.<br>
>><br>
>> 2. Why keeping the old widget around then ?<br>
>> Due to Qt limitations, this new implementation does not support very well<br>
>> being a native widget.<br>
>> But native widget are sometimes mandatory, for example within QScrollArea<br>
>> and QMDIArea, so the QVTKOpenGLSimpleWidget should be used when in needs of<br>
>> VTK rendering in the contact of Qt native widget.<br>
>><br>
>> Also it allows users to switch back to the old widget if necessary.<br>
>><br>
>> 3. I'm not sure what native widgets are, what should I do in my<br>
>> application ?<br>
>><br>
>> Here are the different situation :<br>
>><br>
>> Your Qt application only uses a central QVTKOpenGLWidget for rendering:<br>
>> -> Nothing to do, just build with last master and make sure all is working<br>
>> well<br>
>> Your Qt application only uses QVTKOpenGLWidget within QScrollArea or<br>
>> QMDIArea, or manually set widgets to be native and you are not interested by<br>
>> stereo rendering.<br>
>>  -> Change all your QVTKOpenGLWidget to QVTKOpenGLSimpleWidget and you are<br>
>> good to go<br>
>> Your application uses a non-native QVTKOpenGLWidget for rendering as well<br>
>> as native QVTKOpenGLWidget for rendering (eg: ParaView, with the central<br>
>> rendering widget and the color map editor rendering widget in scroll areas)<br>
>> :<br>
>>  -> Use QVTKOpenGLWidget for non-native widgets and QVTKOpenGLSimpleWidget<br>
>> for native ones. The later will never support stereo.<br>
>><br>
>><br>
>> 4. I followed your recommendation but I see some strange<br>
>> stuff/bugs/rendering issues<br>
>> Even if this new class has been tested extensively and will be used in the<br>
>> next ParaView release, It may still contains some issues. Feel free to<br>
>> discuss them in this mailing list or on our gitlab.<br>
>><br>
>> Best regards,<br>
>><br>
>> Mathieu Westphal<br>
><br>
><br>
> _______________________________________________<br>
> Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
><br>
> Visit other Kitware open-source projects at<br>
> <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
><br>
> Please keep messages on-topic and check the VTK FAQ at:<br>
> <a href="http://www.vtk.org/Wiki/VTK_FAQ" rel="noreferrer" target="_blank">http://www.vtk.org/Wiki/VTK_FAQ</a><br>
><br>
> Search the list archives at: <a href="http://markmail.org/search/?q=vtkusers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=vtkusers</a><br>
><br>
> Follow this link to subscribe/unsubscribe:<br>
> <a href="https://public.kitware.com/mailman/listinfo/vtkusers" rel="noreferrer" target="_blank">https://public.kitware.com/mailman/listinfo/vtkusers</a><br>
><br>
</blockquote></div>