ytree-project / ytree

A yt-based merger-tree code.
Other
15 stars 10 forks source link

parallel_trees function does not work with non-root trees #115

Closed brittonsmith closed 2 years ago

brittonsmith commented 3 years ago

Bug report

The parallel_trees function assumes that all trees provided are roots of their respective forests. Within the code, the _arbor_index attribute is accessed, which only exists for trees that are forest roots. A fix for this needs to take care of cases where multiple trees from the same forest are given. If we are not careful, results could overwrite each other when recombined.

brittonsmith commented 2 years ago

Resolved in PR #123.