<div dir="ltr">Hi Nathan,<div><br></div><div>Ok, I can do that.  My concern was that the active function may not necessarily be the one that the developer meant to keep, but is just the one that happens to be getting called.  However, it's probably a safe assumption that the active function is the correct one, so I'll do that.</div><div><br></div><div>Britton</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Oct 14, 2014 at 4:38 PM, Nathan Goldbaum <span dir="ltr"><<a href="mailto:nathan12343@gmail.com" target="_blank">nathan12343@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Britton,<div><br></div><div>You can tell for yourself by putting a breakpoint in each and then calling the function. You'll end up in a breakpoint inside the "active" function, so you can delete the other one.<span></span></div><div><br></div><div>Thanks for taking care of this!</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Nathan</div></font></span><div class="HOEnZb"><div class="h5"><div><br>On Tuesday, October 14, 2014, Britton Smith <<a href="mailto:brittonsmith@gmail.com" target="_blank">brittonsmith@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div><br></div><div>I was just looking through yt/data_object/selection_data_containers.py and noticed that there are two nearly identical YTCuttingPlaneBase.to_frb functions.  They are almost the same with just a different kwarg and one other line.  This may have been a bad merge.  Can someone who is familiar with this area take a quick look and either get rid of one or tell me which one should be removed?</div><div><br></div><div>Thanks,</div><div>Britton</div></div>
</blockquote></div>
</div></div><br>_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org">yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
<br></blockquote></div><br></div>