bitwalker / bitwalker/libgraph

Poor performance of Graph.replace_vertex/2 for large graphs

Abierto
#85 0 comentarios 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
Elixir
Estrellas
571
Forks
76
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

Similar to #80 with likely similar fix. ~25x speedup for graph with 1M vertices/edges in iex for:
```
g = (Graph.new() |> Graph.add_edges(Enum.map(1..1_000_000, fn i -> {i, 1_000_000 - i} end)))
:timer.tc(fn -> Graph.replace_vertex(g, Enum.random(1..1_000_000), 1_000_001) end, :millisecond)
# {56, #Graph}
:timer.tc(fn -> Graph.replace_vertex_old(g, Enum.random(1..1_000_000), 1_000_001) end, :millisecond)
# {1399, #Graph}
```

Will add to PR #84 as related issues. Would like someone to double check that I haven't mixed up the in-edges and out-edges definitions.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Comienza comparando Graph.replace_vertex/2 con Graph.replace_vertex_old/2 y revisa el issue #80 y el PR #84 para consultar la corrección relacionada. Reproduce el benchmark de IEx en un grafo con 1.000.000 de vértices y aristas y, después, verifica las definiciones de in-edge y out-edge. Se considera terminado cuando el reemplazo es sustancialmente más rápido sin mezclar esos conjuntos de aristas.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
elixir
Área
performance
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.