[API] Add simplified overloads for algorithms (starting with Dijkstra)
- Lingua principale
- C++
- Stelle
- 392
- Fork
- 239
- Merge medio
- 1g 11m
- PR unite (30g)
- 20
Descrizione
## Problem
I am reworking the documentation, and in need of a nice example for the landing page. The most popular algorithm is Dijkstra. However, given that named parameters should be deprecated, illustrating the current API for Dijkstra shortest path algorithm would require full positional argument list and makes the alternative to named parameters rather painful:
```cpp
dijkstra_shortest_paths(g, vertex(0, g),
make_iterator_property_map(pred.begin(), index),
make_iterator_property_map(dist.begin(), index),
costs,
index,
std::less(),
std::plus(),
std::numeric_limits::max(),
0,
dijkstra_visitor());
```
Users expect to write simpler things, like:
- `dijkstra_shortest_paths(g, s, pred, dist, weight);`
- `dijkstra_shortest_paths(g, s, pred, dist, weight, index);` with custom index map when `VertexList=listS`
## Proposal
Add two overloads to cover most use cases. Arity 5 overload should cover 90% of use cases.
```
Arity 3: (g, s, named_params) // existing, to be dropped
Arity 5: (g, s, pred, dist, weight) // NEW, safe
Arity 6: (g, s, pred, dist, weight, ix) // NEW, safe
Arity 11: (g, s, pred, dist, weight, ix, cmp, combine, inf, zero, vis) // existing
Arity 12: (g, s, pred, dist, weight, ix, cmp, combine, inf, zero, vis, color) // existing
```
## List of candidate algorithms to benefit from a better overload set
| Algorithm | Min positional arity | Proposed min | Saved |
|-----------|---------------------|-------------|-------|
| `dijkstra_shortest_paths` | 11 | 5 | 6 |
| `dag_shortest_paths` | 11 | 6 | 5 |
| `bellman_ford_shortest_paths` | 8 | 5 | 3 |
| `astar_search` | 14 | 5 | 9 |
| `prim_minimum_spanning_tree` | 7 | 2 | 5 |
| `johnson_all_pairs_shortest_paths` | 5 | 4 | 1 |
| `breadth_first_search` | 5 | 2 | 3 |
| `depth_first_search` | 3 | 1 | 2 |
Guida per i contributori
Apri la guida per i contributori
Valutazione
Questa issue non è ancora stata valutata.