Fix matrix

[rmflight]

fixes a logical test in the .matrix method, and allows passing more parameters to the underlying igraph methods.

[moritzpschwarz]

Hi @thomasp85 just for your information, I encountered the same issue, see here for a description:

https://stackoverflow.com/questions/76952759/error-in-tidygraph-as-tbl-graph-mutliple-logicals-compared-to-single

and a quick example:

library(tidygraph)
adj <- structure(c(0, 0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 1, 0, 0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 0, 1, 1, 0, 2, 0, 
                   0, 0, 0, 0, 1, 0, 0, 0, 0, 0, 2, 2, 0, 0, 0, 0, 0, 1, 0, 0, 1, 
                   0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 
                   0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0), 
                 dim = c(15L, 15L), 
                 dimnames = list(paste0("v",1:15), 
                                 paste0("v",1:15)))

adj

tidygraph::as_tbl_graph(adj)

The fixes by @rmflight on this branch worked for me!

[moritzpschwarz]

Small update to my issue described above, this has worked as a workaround:

adj %>%
    igraph::graph_from_adjacency_matrix(weighted = TRUE) %>%
    tidygraph::as_tbl_graph()

[thomasp85]

matrix("Thanks", ncol = 3, nrow = 2)
#>      [,1]     [,2]     [,3]    
#> [1,] "Thanks" "Thanks" "Thanks"
#> [2,] "Thanks" "Thanks" "Thanks"