Activity
jetzt habe ich es endlich geschafft auf die Hinweise und Ergänzungen zum Hoodselector einzugehen und noch einmal über den Fallback Mode zu schauen. Dazu habe ich in einem ersten Schritt versucht systematisch Testcases für alle möglichen Anwendungssituationen zu erzeugen. Dadurch können wir prüfen, ob der Hoodselector in jeder von uns vorgesehenen Situation genau das tut was wir erwarten.
Vorweg zu den Begrifflichkeiten:
- Default Hood (Hood ohne Geodaten)
- Real Hood (Hood mit Geodaten)
Die Testcases liegen hier (gerne auf Vollständigkeit prüfen): https://pad.ffnw.de/p/hoods-testcases
Aus den Testcases ergeben sich drei Level der Hooderkennung von denen zwei den Fallback Mode bilden:
- Normal Mode (Router hat Position)
- Scan Mode (Fallback falls Normal Mode fehlschlägt)
- Defaul Hood Mode (Fallback falls Scan Mode ebenfalls fehlschlägt)
Im Code sieht das dann so aus (branch hoodselector-proposal): https://git.nordwest.freifunk.net/ffnw/packages/blob/hoodselector-proposal/hoodselector/files/lib/ffnw/hoodselector/hoodselector
Das ist leider nur Pseudocode mit vielen Kommentaren, da ich bisher noch nichts mit LUA zu tun hatte. Insbesondere folgende Methoden müssten noch implementiert werden (Gerüst und Doku sind im Code schon angegeben):
- get_bssid_list
- sort_bssid_list
- get_hood_by_bssid_list
- get_default_hood
Damit sollte der Hoodselector dann alle Testcases abbilden und dabei immer die optimale Lösung treffen, ohne dass dabei Seiteneffekte entstehen.
Würde mich freuen, wenn ihr euch das ansehen könntet :)
Ich bin den code jetzt noch ein mal durch gegangen und hab den einmal entschlakt. Ich denke das Ticket kann mit dem commit geschlossen werde. f4a7ab38