<div dir="ltr"><div class="gmail_extra">Hi Jon,</div><div class="gmail_extra"><br></div><div class="gmail_extra">The charts related changed look good to me, just added +2 for that piece for both. Thanks for fixing these up, it looks like there are some dashboard issues hopefully they can be cleared up (look unrelated to the charts changes to me).</div><div class="gmail_extra"><br></div><div class="gmail_extra">Marcus<br>
<br><div class="gmail_quote">On Tue, May 9, 2017 at 5:21 PM, Jon Haitz Legarreta <span dir="ltr"><<a href="mailto:jhlegarreta@vicomtech.org" target="_blank">jhlegarreta@vicomtech.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Marcus,<br></div>thanks !<br clear="all"><div><div><div class="gmail_extra"><br>The issues with vtk::PlotSurface were addressed here:<br><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/2787" target="_blank">https://gitlab.kitware.com/<wbr>vtk/vtk/merge_requests/2787</a><br><br>Still pending some approval/review/suggestion, since the build errors seem unrelated to me.<br><br>The last bunch of warnings (including that one belonging to the vtk::vtkScatterPlotMatrix class) is addressed here:<br><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/2799" target="_blank">https://gitlab.kitware.com/<wbr>vtk/vtk/merge_requests/2799</a><br><br>I've got to review the errors in this last still. That's why I did not add any reviewers yet. But if they look immediate to you, suggestions are welcome.<br><br>JON HAITZ<br><br><div><div class="m_1615745494180231990gmail-m_-4882149739593758425gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><table style="color:rgb(0,0,0);font-family:"times new roman";font-size:medium" cellspacing="0" cellpadding="0" border="0" align="center"><tbody><tr><td></td></tr></tbody></table><pre style="color:rgb(0,0,0);white-space:pre-wrap">--<br><br></pre><div><div dir="ltr"></div></div></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote"><div><div class="h5">On 9 May 2017 at 20:34, Marcus D. Hanwell <span dir="ltr"><<a href="mailto:marcus.hanwell@kitware.com" target="_blank">marcus.hanwell@kitware.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div class="h5"><span>On Fri, May 5, 2017 at 10:09 AM, Sean McBride <<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>> wrote:<br>
><br>
> On Wed, 3 May 2017 09:25:09 -0600, David Gobbi said:<br>
><br>
> >I don't think suppressing them is a good idea, redefined member variables<br>
> >are strongly indicative of real bugs.<br>
><br>
> Thanks David & Jon!  We are down to 10 now:<br>
><br>
> Charts/Core/vtkPlot3D.h:152: warning: The class 'vtkPlotSurface' defines member variable with name 'Colors' also defined in its parent class 'vtkPlot3D'.<br>
><br>
> Charts/Core/vtkChartMatrix.h:1<wbr>55: warning: The class 'vtkScatterPlotMatrix' defines member variable with name 'Private' also defined in its parent class 'vtkChartMatrix'.<br>
><br>
</span>I missed your original email, I haven't always had time to monitor the<br>
mailing list. Are both of these still open? I can take a look if so,<br>
they should be pretty simple and I likely wrote them, if there are<br>
others in charts I can take a look too.<br>
</div></div><span class=""><div class="m_1615745494180231990gmail-m_-4882149739593758425HOEnZb"><div class="m_1615745494180231990gmail-m_-4882149739593758425h5">______________________________<wbr>_________________<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 <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensou<wbr>rce/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mail<wbr>man/listinfo/vtk-developers</a><br>
<br>
</div></div></span></blockquote></div><br></div></div></div></div>
</blockquote></div><br></div></div>