Hi Nathan,

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.


On Tue, Oct 14, 2014 at 4:38 PM, Nathan Goldbaum <nathan12343@gmail.com> wrote:
Hi Britton,

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.

Thanks for taking care of this!


On Tuesday, October 14, 2014, Britton Smith <brittonsmith@gmail.com> wrote:
Hi all,

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?


yt-dev mailing list