Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Feature parameter screen #32

Merged
merged 15 commits into from

4 participants

@ChristianScherer
Collaborator

Neue Eingabefelder,deren Checks und Integration zum Project implementiert.
Zudem auch neue Abfrage bezüglich der Vorbedingungen ob RandomWalk ausgewählt wurde oder nicht.
Da vorher nur mit getInputType.getStoch / getDeterm gearbeitet werden konnte haben wir hier eine andere komplexität die jetzt zwar implementiert ist aber nicht komplett schön... in den Abendstunden heute aber nicht anders umzusetzen.

Kommentare hab ich versucht alle Aktuell zu halten (zumindest 98% sollten passen, kleine Anpassungen beim Bugfixing könnten noch unkommentiert sein)

ChristianScherer and others added some commits
@ChristianScherer ChristianScherer Parameterscreen UI mit Presenter/Impl soweit schon funktionsfähig.
Next step -> Abspeichern in Project-Objekt und Querchecks implementieren
0cd6219
@ChristianScherer ChristianScherer Project-Object aktualisiert 8ca96e0
@ChristianScherer ChristianScherer Merge von Master 3b299c6
@ChristianScherer ChristianScherer Merge branch 'master' of https://github.com/DHBW-Karlsruhe/businessho…
…rizon2 into feature-ParameterScreen
a607087
@ChristianScherer ChristianScherer WIP - Commit vor merge mit Master 79beeef
@ChristianScherer ChristianScherer Merge branch 'master' into feature-ParameterScreen
WIP
5b68644
@juliushacker juliushacker Fix Initialisierung Basisjahr und Iterationsschritte. 0eb19f5
@ChristianScherer ChristianScherer WIP Vorlaeufig lauffaehige Version - Integration von Abhaengigkeiten vom
Methodenscreen muessen noch eingepflegt werden
a2ad0d5
@ChristianScherer ChristianScherer WIP - nach Merge mit Master d31bcd6
@ChristianScherer ChristianScherer Alle Projektverwendungen eingepflegt.
Muss aber in Verbindung mit Methodenview noch getestet werden
ca9c82d
@ChristianScherer ChristianScherer Neue Fehlerbehandlung WIP 032bfae
@ChristianScherer ChristianScherer Merge branch 'master' into feature-ParameterScreen WIP b788d1c
@juliushacker juliushacker Validierung der Eingaben nach jeder Eingabe, sodass der ContentContainer
aufgrund der bei der Validierung gefeuerten Events immer weiß, ob der
Screen gerade Valid oder Invalid ist und den Weiter-Button
dementsprechend funktional belegen kann.
416d1c4
@ChristianScherer ChristianScherer Merge branch 'master' into feature-ParameterScreen
Hinzufuegen weitere Textfelder fuer den RandomWalk Prozess 
WIP!
9921a2c
@ChristianScherer ChristianScherer Merge branch 'master' into feature-ParameterScreen
Einarbeitung weitere Eingabefelder und Abpruefung der Vorbedingung ob
RandomWalk ausgewaehlt wurde
5f8cc37
@extr3m0 extr3m0 commented on the diff
...shorizon2/ui/process/parameter/ParameterViewImpl.java
((140 lines not shown))
+ comboBoxIterations
+ .setDescription("Je h\u00F6her die Anzahl der Wiederholungen, desto genauer wird die Vorhersage.");
+ comboBoxIterations.addListener(new Property.ValueChangeListener() {
+
+ /**
+ * Listener der Wiederholungs(Iterations)-Combobox. Es wird
+ * daraufhin mit dem gewaehlten Wert die entsprechende
+ * Presnter-Methode gerufen
+ *
+ * @author Christian Scherer
+ */
+ private static final long serialVersionUID = 1L;
+
+ @Override
+ public void valueChange(ValueChangeEvent event) {
+ presenter.iterationChosen((int) event.getProperty().getValue());
@extr3m0 Owner
extr3m0 added a note

Das ist nicht direkt ein Fehler, hatte aber mit Timo die Erfahrung gemacht, dass die Property nicht immer richtig/aktuell ist? Falls das spaeter Probleme macht, einfach comboBoxIterations.getValue() verwenden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@juliushacker juliushacker merged commit 980602c into master
@extr3m0
Owner

Sauber!
Fuer die Zukunft bitte nicht bei jeder Datei Strg-Shift-F (Code Formatieren) druecken, weil du sonst jede Zeile zusaetzlich editierst und die eigl Aenderungen nicht mehr ubersichtlich sind und merge Probleme entstehen.

@ChristianScherer
Collaborator

Hm okay für mich ist das normal und gehe davon aus dass es jeder macht - gerade dann sollte es doch vom Mergen besser klappen?

@extr3m0
Owner

Dadurch wird vor Allem die Wahrscheinlichkeit erhoeht, dass 2 Leute die selbe Zeile editiert haben (auch wenn es bei dir nur Leerzeichen-Zeug war) und man somit einen manuellen Merge der Datei machen muss.

Bei Dateien, die du sowieso selbst erstellst, ist das was anderes! ;)

@florian-stier
Collaborator

Unsere Nachfolger sollten sich einen einheitlichen Code Formatter einstellen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 12, 2013
  1. @ChristianScherer

    Parameterscreen UI mit Presenter/Impl soweit schon funktionsfähig.

    ChristianScherer authored
    Next step -> Abspeichern in Project-Objekt und Querchecks implementieren
  2. @ChristianScherer
  3. @ChristianScherer

    Merge von Master

    ChristianScherer authored
  4. @ChristianScherer
Commits on Jan 13, 2013
  1. @ChristianScherer
  2. @ChristianScherer
  3. @juliushacker
  4. @ChristianScherer

    WIP Vorlaeufig lauffaehige Version - Integration von Abhaengigkeiten vom

    ChristianScherer authored
    Methodenscreen muessen noch eingepflegt werden
  5. @ChristianScherer
  6. @ChristianScherer

    Alle Projektverwendungen eingepflegt.

    ChristianScherer authored
    Muss aber in Verbindung mit Methodenview noch getestet werden
Commits on Jan 16, 2013
  1. @ChristianScherer
Commits on Jan 17, 2013
  1. @ChristianScherer
Commits on Jan 18, 2013
  1. @juliushacker

    Validierung der Eingaben nach jeder Eingabe, sodass der ContentContainer

    juliushacker authored
    aufgrund der bei der Validierung gefeuerten Events immer weiß, ob der
    Screen gerade Valid oder Invalid ist und den Weiter-Button
    dementsprechend funktional belegen kann.
  2. @ChristianScherer

    Merge branch 'master' into feature-ParameterScreen

    ChristianScherer authored
    Hinzufuegen weitere Textfelder fuer den RandomWalk Prozess 
    WIP!
Commits on Jan 19, 2013
  1. @ChristianScherer

    Merge branch 'master' into feature-ParameterScreen

    ChristianScherer authored
    Einarbeitung weitere Eingabefelder und Abpruefung der Vorbedingung ob
    RandomWalk ausgewaehlt wurde
Something went wrong with that request. Please try again.