[Paraview] endUndoSet called without a beginUndoSet.
Utkarsh Ayachit
utkarsh.ayachit at kitware.com
Tue May 5 15:28:58 EDT 2009
Robert,
What your patch is doing is exactly what I've done, except at the server
manager level (so in vtkSMScalarBarRepresentationProxy instead of
pqScalarBarRepresentation). I guess I am going to mark bug #8958 resolved,
unless someone tells me otherwise :).
Utkarsh
On Tue, May 5, 2009 at 3:14 PM, Dominik Szczerba <dominik at itis.ethz.ch>wrote:
> I know what the reason is.
> I have originally submitted the bug under 'Paraview' project, then
> realizing that the last posts before were the previous year, and in addition
> seeing also project 'Paraview3', I concluded that the former must be
> referring to 2.x branch or so - and I resubmitted to the latter, where also
> the earlier posts were just days before.
> To some extent I may be called guilty, but the 2 actual Paraview projects
> are very confusing.
> Many thanks - and sorry. I only hope you two can agree on a better
> solution. I can test whatever you conclude with.
> thanks and regards,
> Dominik
>
>
> Robert Maynard wrote:
>
>> Odly enough I just finished fixing this bug today, with a patch for the
>> visibility issue at http://www.vtk.org/Bug/view.php?id=8958
>>
>> The locking bug it self can be fixed by simply changing one line.
>>
>> Index: C:/Work/ParaViewGeo/VTK/Widgets/vtkBorderWidget.cxx
>> ===================================================================
>> --- C:/Work/ParaViewGeo/VTK/Widgets/vtkBorderWidget.cxx (revision 3953)
>> +++ C:/Work/ParaViewGeo/VTK/Widgets/vtkBorderWidget.cxx (revision 3954)
>> @@ -275,7 +275,9 @@
>> reinterpret_cast<vtkBorderRepresentation*>(self->WidgetRep)->MovingOff();
>>
>> // stop adjusting
>> - self->EventCallbackCommand->SetAbortFlag(1);
>> + //by setting the abort flag to 0 we allow the end interaction event
>> to be
>> + //caught be other observers so that mouse buttons don't get stuck
>> being down.
>> + self->EventCallbackCommand->SetAbortFlag(0);
>> self->EndInteraction();
>> self->InvokeEvent(vtkCommand::EndInteractionEvent,NULL);
>> }
>>
>> Dominik Szczerba wrote:
>>
>>> Many thanks for such a quick reaction!
>>> I will give it a try next week.
>>> regards,
>>> Dominik
>>>
>>> Utkarsh Ayachit wrote:
>>>
>>>> Dominik,
>>>> I just committed a fix for this issue. Feel free to give it a try and
>>>> let me know if it still doesn't work.
>>>>
>>>> Thanks,
>>>> Utkarsh
>>>>
>>>> /cvsroot/ParaView3/ParaView3/Qt/Core/pqScalarBarRepresentation.cxx,v
>>>> <-- Qt/Core/pqScalarBarRepresentation.cxx
>>>> new revision: 1.11; previous revision: 1.10
>>>> /cvsroot/ParaView3/ParaView3/Servers/ServerManager/Resources/rendering.xml,v
>>>> <-- Servers/ServerManager/Resources/rendering.xml
>>>> new revision: 1.345; previous revision: 1.344
>>>> /cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMScalarBarWidgetRepresentationProxy.cxx,v
>>>> <-- Servers/ServerManager/vtkSMScalarBarWidgetRepresentationProxy.cxx
>>>> new revision: 1.10; previous revision: 1.9
>>>> /cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMScalarBarWidgetRepresentationProxy.h,v
>>>> <-- Servers/ServerManager/vtkSMScalarBarWidgetRepresentationProxy.h
>>>> new revision: 1.6; previous revision: 1.5
>>>>
>>>>
>>>> On Wed, Apr 15, 2009 at 1:49 PM, Dominik Szczerba <dominik at itis.ethz.ch<mailto:
>>>> dominik at itis.ethz.ch>> wrote:
>>>>
>>>> -----BEGIN PGP SIGNED MESSAGE-----
>>>> Hash: SHA1
>>>>
>>>> Even simpler way to reproduce:
>>>>
>>>> just move the (sliced) dataset with MMB onto the visible colorbar: a
>>>> lock results for me. Invisible datasets hide their colorbars, but
>>>> they
>>>> are somehow still there, locking the navigation: a clear bug, at
>>>> least
>>>> on my system.
>>>>
>>>> - - Dominik
>>>>
>>>> Dominik Szczerba wrote:
>>>> > No, it does not have to do with the mouse outside the render
>>>> window.
>>>> > This is a bug with colorbar. To reproduce:
>>>> >
>>>> > Load a dataset, slice it, color it with a scalar, display the
>>>> colorbar.
>>>> > Load another dataset, slice it, color it.
>>>> > Now hide the first slice (unclick the eye).
>>>> > The color legend is gone, but stays there hidden, faking the
>>>> mouse. Move
>>>> > the cursor to see false mouse-over effects. This now often results
>>>> in
>>>> > the mentioned "locking" while dragging with the middle button. It
>>>> is the
>>>> > invisible colorbar that gets moved instead of the dataset and
>>>> the whole
>>>> > thing gets eventually stuck. It is extremely annoying, if not
>>>> impossible
>>>> > in case of complex pipeline states, to go through all entries and
>>>> switch
>>>> > off the colorbars. Is it possible to fix this?
>>>> >
>>>> > - Dominik
>>>> >
>>>> > Utkarsh Ayachit wrote:
>>>> >> That generally happens when you start dragging but move the
>>>> mouse out
>>>> >> of the renderwindow before releasing the mouse button. Is that
>>>> indeed
>>>> >> the case?
>>>> >
>>>> >> Utkarsh
>>>> >
>>>> >> On Tue, Apr 14, 2009 at 6:19 AM, Dominik Szczerba
>>>> <dominik at itis.ethz.ch <mailto:dominik at itis.ethz.ch>> wrote:
>>>> >> Hi,
>>>> >> I am getting annoying critical problems when moving datasets
>>>> (here, a
>>>> >> result of cutting tool): the dataset keeps moving even after I
>>>> release
>>>> >> the middle mouse button and I have to click around blind like
>>>> crazy to
>>>> >> "unlock" it. I see the following in the output window:
>>>> >
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >> endUndoSet called without a beginUndoSet.
>>>> >
>>>> >> Can someone please advise how to fix it?
>>>> >
>>>> >> regards,
>>>> >> Dominik
>>>> > _______________________________________________
>>>> > Powered by www.kitware.com <http://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://www.paraview.org/mailman/listinfo/paraview
>>>> >
>>>>
>>>> - --
>>>> Dominik Szczerba, PhD
>>>> Computational Physics Group
>>>> IT'IS Foundation for Research
>>>> www.itis.ethz.ch <http://www.itis.ethz.ch>
>>>> -----BEGIN PGP SIGNATURE-----
>>>> Version: GnuPG v2.0.9 (GNU/Linux)
>>>> Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org
>>>>
>>>> iEYEARECAAYFAknmHiYACgkQ/EBMh9bUuzJwwQCfZSHSpQ99zTAxS6tiIYAhZyvm
>>>> DqgAoKFZHkNGdmIro+9P4Zel4P/U7eC4
>>>> =d9xA
>>>> -----END PGP SIGNATURE-----
>>>>
>>>>
>>>>
>>>
>>
> --
> Dominik Szczerba, PhD
> Computational Physics Group
> IT'IS Foundation
> http://www.itis.ethz.ch
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.paraview.org/pipermail/paraview/attachments/20090505/ce920f9e/attachment.htm>
More information about the ParaView
mailing list