<div dir="ltr"><div class="gmail_extra">Thanks Ben. </div><div class="gmail_extra"><br></div><div class="gmail_extra"><div class="gmail_quote">On Wed, May 8, 2013 at 12:32 PM, Ben Burnett <span dir="ltr"><<a href="mailto:benjam.arlyn@gmail.com" target="_blank">benjam.arlyn@gmail.com</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">Hi Aashish,<div><br></div><div>I reviewed this branch. I tested basic functionality, but I'm not sure how to test the things you actually worked on/added, or if there is a way yet.</div>
</div></blockquote><div><br></div><div style>You can run drawSphere test. That's the new test added to show the functionality   </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr">
<div>

<br></div><div>The code looks decent, a few typos and style issues but nothing major.</div></div></blockquote><div> </div><div style>Can you push fixes for these typos and style issues? I will appreciate that. </div><div>
<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><br></div><div>Ben</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
<div><div>On Tue, May 7, 2013 at 3:26 PM, Aashish Chaudhary <span dir="ltr"><<a href="mailto:aashish.chaudhary@kitware.com" target="_blank">aashish.chaudhary@kitware.com</a>></span> wrote:<br>


</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><div dir="ltr">This branch added support for globe rendering for climatepipes. It does not actually provide means to draw the globe yet but added new transformations and a test to support globe rendering for the next branch which I am working on now. <div>




<br></div><div>It would be nice if someone can approve this branch. </div><div><br></div><div>Thanks</div><span><font color="#888888"><div><br clear="all"><div><br></div>-- <br>| Aashish Chaudhary <br>| R&D Engineer         <br>



| Kitware Inc.            <br>
| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a>    
</div></font></span></div>
<br></div></div>_______________________________________________<br>
Opengeoscience-developers mailing list<br>
<a href="mailto:Opengeoscience-developers@public.kitware.com" target="_blank">Opengeoscience-developers@public.kitware.com</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/opengeoscience-developers</a><br>
<br></blockquote></div><br></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>| Aashish Chaudhary <br>| R&D Engineer         <br>| Kitware Inc.            <br>| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a>    
</div></div>