Skip to content

Commit

Permalink
fix: do not add vertex names to edge_betweenness_subset result
Browse files Browse the repository at this point in the history
as the result is associated with edges and not vertices.
Also fix invalid dependency specification: no dependency on V(g).
  • Loading branch information
szhorvat authored and aviator-bot committed Feb 6, 2024
1 parent 804ec89 commit 32ef7b3
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
4 changes: 1 addition & 3 deletions R/aaa-auto.R
Original file line number Diff line number Diff line change
Expand Up @@ -985,9 +985,7 @@ edge_betweenness_subset_impl <- function(graph, eids=E(graph), directed=TRUE, so
on.exit( .Call(R_igraph_finalizer) )
# Function call
res <- .Call(R_igraph_edge_betweenness_subset, graph, eids-1, directed, sources-1, targets-1, weights)
if (igraph_opt("add.vertex.names") && is_named(graph)) {
names(res) <- vertex_attr(graph, "name", V(graph))
}

res
}

Expand Down
2 changes: 0 additions & 2 deletions tools/stimulus/functions-R.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -416,8 +416,6 @@ igraph_edge_betweenness_cutoff:
IGNORE: RR

igraph_edge_betweenness_subset:
DEPS: |-
eids ON graph, weights ON graph V(graph), res ON graph V(graph), sources ON graph, targets ON graph

igraph_harmonic_centrality:
IGNORE: RR, RC, RInit
Expand Down

0 comments on commit 32ef7b3

Please sign in to comment.