Re: [Paraview] Q-criterion still wrong ?

2012-08-17 Thread Aurélien Marsan
Thanks to you :-)

Aurélien

2012/8/14 Andy Bauer 

> FYI: The fix has made its way into ParaView. Thanks again for the bug
> report.
>
> Andy
>
>
> On Fri, Jul 20, 2012 at 2:48 PM, Andy Bauer wrote:
>
>> Thanks for finding this.  I have a fix in gerrit at
>> http://review.source.kitware.com/#/t/957/. It should make it into VTK
>> and ParaView shortly.
>>
>> Andy
>>
>> On Fri, Jul 20, 2012 at 8:53 AM, Aurélien Marsan wrote:
>>
>>>  Good morning,
>>>
>>> I have just found the following bug report about a mistake in the
>>> Q-criterion formula.
>>> http://paraview.org/Bug/view.php?id=12858
>>> that is now closed.
>>>
>>> However, the "1/2" factor may still be missing.
>>> In vtkGradientFilter.cxx, in the VTK/Grahpics directory, on the line 65,
>>> t1 should be divided by 2. (Paraview 3.14.1)
>>>
>>> Should I re-open the bug report ?
>>>
>>> With best regards,
>>>
>>> Aurélien
>>>
>>> ___
>>> 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://www.paraview.org/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://www.paraview.org/mailman/listinfo/paraview


Re: [Paraview] Q-criterion still wrong ?

2012-08-15 Thread Andy Bauer
It's an option, along with vorticity, under the Gradient of Unstructured
Dataset filter. The current behavior is a bit non-intuitive in that if you
check Compute Vorticity the output will be the vorticity (assuming a vector
field), otherwise the output will be the gradient. For the Compute
QCriterion option though it outputs that in addition to the other outputs.
This will get changed in 4.0 though to be a bit clearer.

Andy

On Wed, Aug 15, 2012 at 8:03 AM, Florian Lindner wrote:

> Sorry for being ignorant.. I never found a Q criteria filter in
> paraview? Where is it?
>
> 2012/8/14 Andy Bauer :
> > FYI: The fix has made its way into ParaView. Thanks again for the bug
> > report.
> >
> > Andy
> >
> > On Fri, Jul 20, 2012 at 2:48 PM, Andy Bauer 
> wrote:
> >>
> >> Thanks for finding this.  I have a fix in gerrit at
> >> http://review.source.kitware.com/#/t/957/. It should make it into VTK
> and
> >> ParaView shortly.
> >>
> >> Andy
> >>
> >> On Fri, Jul 20, 2012 at 8:53 AM, Aurélien Marsan 
> >> wrote:
> >>>
> >>> Good morning,
> >>>
> >>> I have just found the following bug report about a mistake in the
> >>> Q-criterion formula.
> >>> http://paraview.org/Bug/view.php?id=12858
> >>> that is now closed.
> >>>
> >>> However, the "1/2" factor may still be missing.
> >>> In vtkGradientFilter.cxx, in the VTK/Grahpics directory, on the line
> 65,
> >>> t1 should be divided by 2. (Paraview 3.14.1)
> >>>
> >>> Should I re-open the bug report ?
> >>>
> >>> With best regards,
> >>>
> >>> Aurélien
> >>>
> >>> ___
> >>> 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://www.paraview.org/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://www.paraview.org/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://www.paraview.org/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://www.paraview.org/mailman/listinfo/paraview


Re: [Paraview] Q-criterion still wrong ?

2012-08-15 Thread Florian Lindner
Sorry for being ignorant.. I never found a Q criteria filter in
paraview? Where is it?

2012/8/14 Andy Bauer :
> FYI: The fix has made its way into ParaView. Thanks again for the bug
> report.
>
> Andy
>
> On Fri, Jul 20, 2012 at 2:48 PM, Andy Bauer  wrote:
>>
>> Thanks for finding this.  I have a fix in gerrit at
>> http://review.source.kitware.com/#/t/957/. It should make it into VTK and
>> ParaView shortly.
>>
>> Andy
>>
>> On Fri, Jul 20, 2012 at 8:53 AM, Aurélien Marsan 
>> wrote:
>>>
>>> Good morning,
>>>
>>> I have just found the following bug report about a mistake in the
>>> Q-criterion formula.
>>> http://paraview.org/Bug/view.php?id=12858
>>> that is now closed.
>>>
>>> However, the "1/2" factor may still be missing.
>>> In vtkGradientFilter.cxx, in the VTK/Grahpics directory, on the line 65,
>>> t1 should be divided by 2. (Paraview 3.14.1)
>>>
>>> Should I re-open the bug report ?
>>>
>>> With best regards,
>>>
>>> Aurélien
>>>
>>> ___
>>> 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://www.paraview.org/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://www.paraview.org/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://www.paraview.org/mailman/listinfo/paraview


Re: [Paraview] Q-criterion still wrong ?

2012-08-14 Thread Andy Bauer
FYI: The fix has made its way into ParaView. Thanks again for the bug
report.

Andy

On Fri, Jul 20, 2012 at 2:48 PM, Andy Bauer  wrote:

> Thanks for finding this.  I have a fix in gerrit at
> http://review.source.kitware.com/#/t/957/. It should make it into VTK and
> ParaView shortly.
>
> Andy
>
> On Fri, Jul 20, 2012 at 8:53 AM, Aurélien Marsan wrote:
>
>> Good morning,
>>
>> I have just found the following bug report about a mistake in the
>> Q-criterion formula.
>> http://paraview.org/Bug/view.php?id=12858
>> that is now closed.
>>
>> However, the "1/2" factor may still be missing.
>> In vtkGradientFilter.cxx, in the VTK/Grahpics directory, on the line 65,
>> t1 should be divided by 2. (Paraview 3.14.1)
>>
>> Should I re-open the bug report ?
>>
>> With best regards,
>>
>> Aurélien
>>
>> ___
>> 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://www.paraview.org/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://www.paraview.org/mailman/listinfo/paraview


Re: [Paraview] Q-criterion still wrong ?

2012-07-20 Thread Andy Bauer
Thanks for finding this.  I have a fix in gerrit at
http://review.source.kitware.com/#/t/957/. It should make it into VTK and
ParaView shortly.

Andy

On Fri, Jul 20, 2012 at 8:53 AM, Aurélien Marsan wrote:

> Good morning,
>
> I have just found the following bug report about a mistake in the
> Q-criterion formula.
> http://paraview.org/Bug/view.php?id=12858
> that is now closed.
>
> However, the "1/2" factor may still be missing.
> In vtkGradientFilter.cxx, in the VTK/Grahpics directory, on the line 65,
> t1 should be divided by 2. (Paraview 3.14.1)
>
> Should I re-open the bug report ?
>
> With best regards,
>
> Aurélien
>
> ___
> 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://www.paraview.org/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://www.paraview.org/mailman/listinfo/paraview


[Paraview] Q-criterion still wrong ?

2012-07-20 Thread Aurélien Marsan
Good morning,

I have just found the following bug report about a mistake in the
Q-criterion formula.
http://paraview.org/Bug/view.php?id=12858
that is now closed.

However, the "1/2" factor may still be missing.
In vtkGradientFilter.cxx, in the VTK/Grahpics directory, on the line 65, t1
should be divided by 2. (Paraview 3.14.1)

Should I re-open the bug report ?

With best regards,

Aurélien
___
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://www.paraview.org/mailman/listinfo/paraview