I was looking at the code in Tachyon and I think I've found something that isn't quite right (or at least is misleading and needs to be documented?)

In theStackTraceCollector, it seems likeself.skip_idle is set but never used, as everycollect method gets anotherskip_idle parameter that they use in_iter_stacks (or their calls to parent class' methods).

Based on thecode for e.g.live_collector, it seems like the pattern is to use the attribute, not a parameter.

Linked PRs

• gh-145458: useself.skip_idle #145459