Fix branchedResetPropagation

BREAKING CHANGE - Forces branches to be directed

Signed-off-by: nif <nif@incremental.social>
This commit is contained in:
Nif 2024-02-12 19:46:31 +00:00
parent 4f807aaf96
commit 5e32fa4985

View file

@ -338,34 +338,21 @@ export const branchedResetPropagation = function (
tree: GenericTree, tree: GenericTree,
resettingNode: GenericTreeNode resettingNode: GenericTreeNode
): void { ): void {
const visitedNodes = [resettingNode]; const links = unref(tree.branches);
let currentNodes = [resettingNode]; if (links === undefined) return;
if (tree.branches != null) { let reset: GenericTreeNode[] = [];
const branches = unref(tree.branches); let current = [resettingNode];
while (currentNodes.length > 0) { while (current.length != 0) {
const nextNodes: GenericTreeNode[] = []; let next: GenericTreeNode[] = [];
currentNodes.forEach(node => { for (let node of current) {
branches for (let link of links.filter(link => link.startNode === node)) {
.filter(branch => branch.startNode === node || branch.endNode === node) if ([...reset, ...current].includes(link.endNode)) continue
.map(branch => { next.push(link.endNode);
if (branch.startNode === node) { link.endNode.reset?.reset();
return branch.endNode; }
} };
return branch.startNode; reset = reset.concat(current);
}) current = next;
.filter(node => !visitedNodes.includes(node))
.forEach(node => {
// Check here instead of in the filter because this check's results may
// change as we go through each node
if (!nextNodes.includes(node)) {
nextNodes.push(node);
node.reset?.reset();
}
});
});
currentNodes = nextNodes;
visitedNodes.push(...currentNodes);
}
} }
}; };