[yt-dev] Clump finder in 3.0

[Matthew Turk]

Hi Nathan,

On Sun, Jun 29, 2014 at 5:31 PM, Nathan Goldbaum <nathan12343 at gmail.com> wrote:
> Hi all,
>
> I'm taking a look at the plotting docs right now.
>
> One of the scripts in the callback documentation currently fails with a
> NotImplementedError.

Ah, this is a side-effect of pf.h not having the .grids attribute;
simple fix.  The clump finder should work -- I completely rewrote the
it for 3.0.  There is a lingering issue with the way clump IDs are
assigned, which was the subject of a failed PR:

https://bitbucket.org/yt_analysis/yt/pull-request/717/be-much-more-careful-about-assigning-clump

The issue is that the clump IDs don't quite match up for some types of
data, particularly multiple root grids.  I have some scripts here to
verify it.

>
> The somewhat scipt is here: http://paste.yt-project.org/show/4857/
>
> And the traceback is here: http://paste.yt-project.org/show/4858/
>
> I'm not sure whether anyone has looked in detail at the clump finder in 3.0.
> Does anyone know about its status?  I'm not even sure if this is an issue
> with the clump finder per se.
>
> -Nathan
>
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>