PoonLab / Kaphi

Kernel-embedded ABC-SMC for phylodynamic inference
GNU Affero General Public License v3.0
4 stars 2 forks source link

confirm that branch lengths and topology in igraph object is correctly passed from R to C #7

Closed ArtPoon closed 8 years ago

ArtPoon commented 8 years ago

implement this in a unit test suite

ArtPoon commented 8 years ago

Note that for a graph that has been converted from R to C witih R_SEXP_to_igraph, we cannot use the C-based attribute handling functions. This is why I was unable to detect the length attribute set in R until I switched to using R_igraph_attribute_has_attr. See this forum thread.

ArtPoon commented 8 years ago

Problem trying to call R_igraph_attribute_get_numeric_edge_attr():

Error at iterators.c:1847 :Cannot create iterator, invalid selector, Invalid value

We're passing an instance of igraph_es_t as an argument to this function, but apparently it hasn't been initialized correctly.