Skip to content

Commit

Permalink
[fud2] Fix planner so that it doesn't rely on the order operations ar…
Browse files Browse the repository at this point in the history
…e defined (#2211)

There was a small bug with the planner where it would stop searching
when a through op appeared anywhere in the `ops` map. It really should
only stop if an op is one edge away from the current state. This worked
before because I guess the `ops` map happened to be ordered in such a
way where things worked.

Fixes #2209.
  • Loading branch information
sgpthomas committed Jul 18, 2024
1 parent 757d77f commit f320a21
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions fud2/fud-core/src/exec/planner.rs
Original file line number Diff line number Diff line change
Expand Up @@ -92,11 +92,11 @@ impl SingleOpOutputPlanner {
state_queue.push(op.output[0]);
visited[op.output[0]] = true;
breadcrumbs[op.output[0]] = Some(op_ref);
}

// Finish when we reach the goal edge.
if end == Destination::Op(op_ref) {
break;
// Finish when we reach the goal edge.
if end == Destination::Op(op_ref) {
break;
}
}
}
}
Expand Down

0 comments on commit f320a21

Please sign in to comment.