<div dir="ltr">First off, Cameron, I'm glad you're on board with this.  As someone who used that functionality, it's important that you're not left high and dry.<div><br></div><div>The original Halo object had a pretty limited scope and was basically only usable inside the halo finding operations.  I would definitely like to see us move to the new Halo object in yt/analysis_modules/halo_analysis/halo_object.py since it's the one being used with the HaloCatalog.  I think we can also get rid of the HaloFinder class since we are replicating all of its functionality as well.  I think, in general, we should be migrating all halo related functionality into yt/analysis_modules/halo_analysis.</div>
<div><br></div><div>Britton</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Mar 24, 2014 at 4:30 AM, Cameron Hummels <span dir="ltr"><<a href="mailto:chummels@gmail.com" target="_blank">chummels@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">I know you're not asking me, but as a user of the old halo objects, I'm OK with this moving forward.  The only suggestion I have is a short document in the yt 3.0 docs explaining the switch and roughly how to replicate basic functionality.  I think some of that already exists, though.<div>


<br></div><div>Cameron</div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">On Sun, Mar 23, 2014 at 2:46 PM, Matthew Turk <span dir="ltr"><<a href="mailto:matthewturk@gmail.com" target="_blank">matthewturk@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 all,<br>
<br>
With the new halo analysis machinery that Britton, Brian C and Hilary<br>
have been working on getting integrated into mainline, do we still<br>
need the old halo objects machinery?<br>
<br>
It seems to me that if we are able to replicate all the functionality,<br>
*and* call the old halo finders, we don't need the actual code for<br>
"class Halo" and "class HaloFinder" and the like.  This is the stuff<br>
that's mostly in halo_objects.py in yt/analysis_modules/halo_finding/<br>
.<br>
<br>
Britton, Brian and Hilary -- does that sound okay to you?<br>
<br>
-Matt<br>
_______________________________________________<br>
yt-dev mailing list<br>
<a href="mailto:yt-dev@lists.spacepope.org" target="_blank">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>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br>Cameron Hummels<div>Postdoctoral Researcher</div><div>Steward Observatory</div><div>University of Arizona</div>
<div><a href="http://chummels.org" target="_blank">http://chummels.org</a></div>


</font></span></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>