Simon Schölly
Simon Schölly
Interesting - as far as I know, `ArnoldiMethod` is mostly used here for spectral methods - i.e. calculating the eigenvalues of adjacency and laplacian matrices. If we had another (maybe...
The `collect_if_not_vector` also has the downside, that we might unnecessary allocate some things. What we maybe want, is that these methods return something that implements the [Indexing operators](https://docs.julialang.org/en/v1/manual/interfaces/#Indexing). So we...
> Very good question, for the moment, the `collect_if_not_vector` sounds like the best workaround but feels a bit clunky, maybe it can be refined by checking outneighbors implement indexing methods...
Something like ```julia function collect_if_not_indexable(vs) if Base.hasmethod(getindex, (typeof(vs), Integer)) return vs else return collect(vs) end end ```
Hi Jonas, thanks for your contribution. Code reviews here take quite a while at the moment, especially when the reviewer tries to understand the algorithm and we are also a...
You may want to add the `edge_betweenness_centrality` symbol to the list [here](https://github.com/JuliaGraphs/Graphs.jl/blob/d3b2706768672e160d4b00ca4496270ff0561c14/src/Graphs.jl#L75-L425) so that it is made available when someone writes `using Graphs`. Furthermore, you actually need to include the...
I feel like I have seen this question before, but maybe it was about something else. I am not sure if this is a bug, or a deliberate choice to...
The helper function `vf2match!` is recursive. `vf2` does nothing more than trying out all permutations of vertices with some pruning rules to speed up things. So the general complexity is...
[Here](https://stackoverflow.com/questions/71956946/how-to-increase-stack-size-for-julia-in-windows#answer-71991634) they describe a way to set the stack size for a Task, and run the computation in said Task. Perhaps you could also try that? If it works, I...
Thanks, while such graphs could certainly be interesting, we try to keep the number of dependencies of this package as low as possible. Therefore the only way I see here...