JosiahParry / sfdep

A tidy interface for spatial dependence
https://sfdep.josiahparry.com/
GNU General Public License v3.0
124 stars 5 forks source link

set operations should sort neighbour indices #54

Closed rsbivand closed 3 months ago

rsbivand commented 3 months ago

In my rev-dep checks, nb_union failed on print because the neighbour indices were not sorted in ascending order. In print.nb, n.comp.nb was run, and chose the igraph path, so needing coercion to a sparse matrix then a graph - coercion to a sparse matrix requires indices to be given in ascending order.

netlify[bot] commented 3 months ago

Deploy Preview for elastic-lumiere-271348 canceled.

Name Link
Latest commit b92b86860a39aa525c04caeb309c272cd36786a4
Latest deploy log https://app.netlify.com/sites/elastic-lumiere-271348/deploys/666c953c97987c00082f17f9
JosiahParry commented 3 months ago

Thank you!!