[Paraview-developers] Help with fixing a bug in one of the plugins

[Mohammad A.Raji]

Thank you. You're right, I was using the call for something else when I
noticed it.

Best,
Mo

On 30 May 2016 at 02:32, Joachim Pouderoux <joachim.pouderoux at kitware.com>
wrote:

> Hello,
>
> Absolutely, the GaussianPoint plugin comes with ParaView sources so you
> can follow notes in CONTRIBUTING.md
>
> Regarding the reported issue, it seems like you are right: height and
> width are reversed. However, I have not been able to found any call to this
> helper function (AddGaussianControlPoint).
>
> Best regards,
> Joachim
>
> *Joachim Pouderoux*
>
> *PhD, Technical Expert*
> *Kitware SAS <http://www.kitware.fr>*
>
>
> 2016-05-30 6:56 GMT+02:00 Mohammad A.Raji <mahmadza at vols.utk.edu>:
>
>> Hello all,
>>
>> I'm quite new to the Paraview world and I believe I've noticed a bug in
>> the PointSprite plugin. Is the pull request procedure the same for plugins
>> as well? (Similar to
>> https://github.com/Kitware/ParaView/blob/master/CONTRIBUTING.md?)
>>
>> It would also be wonderful if someone could take a look at the issue just
>> so that I know I haven't gone insane.
>>
>> In /Plugins/PointSprite/Graphics/vtk1DGaussianTransferFunction.cxx:186
>> the function AddGaussianControlPoint calls SetGaussianControlPoint but with
>> the wrong order of parameters, incorrectly switching width and height in
>> the end control points tuple. Here's a link to the file on Github:
>> https://github.com/Kitware/ParaView/blob/ed854cdd8b1b0f704d3c6c3d0d2f2a14b882cd92/Plugins/PointSprite/Graphics/vtk1DGaussianTransferFunction.cxx
>> .
>>
>> Regards,
>> Mo
>>
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Search the list archives at:
>> http://markmail.org/search/?q=Paraview-developers
>>
>> Follow this link to subscribe/unsubscribe:
>> http://public.kitware.com/mailman/listinfo/paraview-developers
>>
>>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/paraview-developers/attachments/20160601/d879547a/attachment.html>