InnoZ / MAS_Interface

Apache License 2.0
1 stars 0 forks source link

add buttons for mode selection on od map #45

Closed kjoscha closed 7 years ago

kjoscha commented 7 years ago

So, es werden hier nun beim Request die od-Relationen für alle Modes in einen Hash-Table geschrieben. Buttonklick bewirkt dann das Laden der im Arbeitsspeicher befindlichen Daten.

Das hat den Vorteil, dass nach dem Request keine Ladezeiten beim Wechseln des Modes anfallen, jedoch den Nachteil, dass der Request dementsprechend länger dauert. Da in meinem Verständnis die Szenarien nach der Erstellung aber eigentlich statisch bleiben und damit auch die Pläne und damit auch die OD-Relationen, würde ich im nächsten Schritt die OD-Relationen beim Erstellen des Szenarios speichern (Datenbank oder Datei, mal schauen).

Lieg ich damit richtig? Es sind keine Veränderungen von Szenarien nach der Erstellung angedacht, oder @BenSto?

BenSto commented 7 years ago

Kann ich nicht final beantworten, Denke aber auch das erstellte Szenarien final sind. Alle modes direkt zu laden finde ich gut, allerdings sind schon einzelne modes mit vielen Punkten bzw. Plans schon recht langsam zumindest lokal. Müssen wir mal schauen und ggf testen

BenSto commented 7 years ago

Ja speichern ist der beste Weg, wobei ich da stark für DB wäre anstatt wieder File Wirrwarr ;-)

00Bock commented 7 years ago

die regelmäßig (z.B. einmal im Quartal) erstellten Szenarien sind dann für den entsprechenden Zeitraum statisch.

kjoscha commented 7 years ago

Alle modes direkt zu laden finde ich gut, allerdings sind schon einzelne modes mit vielen Punkten bzw. Plans schon recht langsam zumindest lokal.

jo merke ich auch grad bei den Seed-Szenarien. Dann baue ich jetzt direkt mit Speicherung bei der Szenarienerstellung...

kjoscha commented 7 years ago

gerne mal review @BenSto

BenSto commented 7 years ago

Unabhängig von den paar Kommentaren, sieht das Refactoring und die Neuentwicklung, in meinen Augen, sehr gut und clean aus, teilweise schöne Vereinfachung und Codeeinsparung! 🥇 @kjoscha

kjoscha commented 7 years ago

:nerd_face: