Properly handle cases with zero features. about trackpy HOT 3 CLOSED

I originally posted the same thing in #59

The KDTree hash (class TreeFinder) already does this in some form:

        n = len(self.points)
        if n == 0:
            raise ValueError('Frame (aka level) contains zero points')

Maybe that could be moved a little higher up the call stack, and another check could be added at the yield statement at the end of link_iter().

from trackpy.

As Tom has been arguing, I guess this shouldn't raise, because it is possible to have a frame with no particles. Clearly not in your application, though! :- )

Thanks for pointing it out. I will start by making some tests -- all black frames, a black frame inserted among typical frames, a frame where maxima exist but will all be filtered out during locate, etc. -- and see what breaks where.

from trackpy.

Cool. Maybe we want a global option, HALT_ON_EMPTY_FRAME? Or maybe we could be raising warnings only. Or the global option could have 3 values: halt, warn, and ignore.

nathan

On Feb 6, 2014, at 2:22 PM, Dan Allan [email protected] wrote:

As Tom has been arguing, I guess this shouldn't raise, because it is possible to have a frame no particles. Clearly not in your application, though! :- )

Thanks for pointing it out. I will start by making some tests -- all black frames, a black frame inserted among typical frames, a frame where maxima exist but will all be filtered out during locate, etc. -- and see what breaks where.

—
Reply to this email directly or view it on GitHub.

from trackpy.