bitwalker / bitwalker/libgraph
Poor performance of Graph.replace_vertex/2 for large graphs
- Lingua principale
- Elixir
- Stelle
- 571
- Fork
- 76
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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.
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Direzione di ricerca
Inizia confrontando Graph.replace_vertex/2 con Graph.replace_vertex_old/2 e consulta l’issue #80 e la PR #84 per la correzione correlata. Riproduci il benchmark di IEx su un grafo con 1.000.000 di vertici e archi, quindi verifica le definizioni di in-edge e out-edge. Il lavoro è completato quando la sostituzione è sostanzialmente più veloce senza mescolare questi insiemi di archi.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- elixir
- Ambito
- performance
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100