Agustín Borgna
Agustín Borgna
As John said, I'm available for discussing the quantum-classical part on discord or by email if you need :)
Hi, you can use `Graph::into_edge_type` to temporarily change it to Undirected and run the search.
It was actually removed in 696961bc82341659705c5b2f376c8460c23b43cb in favor of an `IntoNeighborsUndirected` trait (#85) which hasn't been implemented yet.
I think a `with_capacity` constructor for the stack would be reasonable for `Topo`, and probably for the other traversals too so that they have a similar interface.
Hi, sorry for the delay. Adding the extra parameter is a breaking change, as it can break type inference on the call sites (as shown by the failing test). The...
Hi. The CI error is fixed in HEAD. Could you rebase your changes on the latest version ?
Yes, it breaks in rust 1.41 (the MSRV). Replacing `&[...]` with `[...].iter()` should do the trick. You can also test it by changing the toolchain version locally: `rustup override set...
Since you need stable identifiers and parallel edges, did you consider using a [`StableGraph`](https://docs.rs/petgraph/0.6.0/petgraph/stable_graph/struct.StableGraph.html#)? There are plans for updating the GraphMap implementation, I'll add supporting parallel edges to the list...
Nice ! The unsafe block was introduced because it was quite faster than the safe code that was there before (https://github.com/petgraph/petgraph/pull/427#issuecomment-840089975). Did you check how the benchmarks compare with HEAD...
This is great, thanks !