Tamás Nepusz

I think the performance impact would be negligible; `igraph_i_st_vertex_connectivity_check_errors` is called only once at the beginning of `igraph_i_st_vertex_connectivity_undirected()` or íigraph_i_st_vertex_connectivity_directed()`. So, any performance impact would probably be non-negligible only if...

Note to self: the error checks in `igraph_st_vertex_connectivity()` are not in vain when they are called O(N^2) times from `igraph_st_vertex_connectivity()` as we are always testing for different vertex pairs, so...

Good question. I think this is one of those historical artifacts again; back in the early days we didn't have a separate interruption handler and the status / progress handler...

Okay, so to sum it all up: * Printing attributes this way is mostly for testing and debugging purposes. * We already have canonical functions to do that in `test_utilities.c`...

> Can we get this into shape and merge before 0.10? It's not a tragedy if this does not get into 0.10 but only to 0.10.1 :) If you manage...

I think it’s good to go whenever you feel it’s ready for a merge.

The way I see it it seems like we should have some kind of an `igraph_transitivity_local_directed()` function with an extra `igraph_directed_transitivity_method_t` enum argument that allows the user to select a...

Okay, I've read the thread; let me summarize what's being proposed here so we can agree that we are on the same page. Proposal nr. 1 is to stop using...

See the commit referenced above; this one tweaks the setup script so you can now use `IGRAPH_USE_PKG_CONFIG=1` instead of `--use-pkg-config`. I'm about to release a new version soon so I...

I cannot reproduce this issue either on my machine (with Cairo 1.21.0), but it's definitely weird. Trying to reproduce the problem in a clean virtualenv on Windows would definitely help.