You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Fixes#1323
This fix generalises the one from #1191.
SimpleReplace was too eager in disconnecting/connecting edges to the
replacement graph, and that caused issues when querying the neighbours
of multiports.
This gets resolved by delaying all new connections to the replacement
until after we have computed all of them.
We don't need to do explicit disconnections to the replaced subgraph,
since the nodes get removed anyway.
---------
Co-authored-by: Douglas Wilson <141026920+doug-q@users.noreply.github.com>
While trying to replace the two nodes in the middle of this diagram
with this graph
I got an
unwrap
panic on SimpleReplacement::apply:158I wrote a pure-hugr failing test in
hugr/hugr-core/src/hugr/rewrite/simple_replace.rs
Lines 720 to 721 in 4e0ad64
The text was updated successfully, but these errors were encountered: