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

[Joachim Pouderoux]

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/20160530/0f8d3153/attachment.html>