Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Testen #10

Closed
TanDro opened this issue Oct 1, 2018 · 3 comments
Closed

Testen #10

TanDro opened this issue Oct 1, 2018 · 3 comments
Assignees
@jhalmen

Comments

@TanDro
Copy link
Contributor

TanDro commented Oct 1, 2018

@jhalmen könntest du bitte den letzten Commit 5b38946 testen!
@TanDro
Copy link
Contributor Author

TanDro commented Oct 1, 2018

Meinte: 4e2dcb0

@jhalmen
Copy link
Contributor

jhalmen commented Oct 2, 2018

die commit message von 4e2dcb0 ist irreführend, ich hatte in dem Plot Labels zu den verschiedenen Datenpunkten erwartet.
da es nur im Daten-Fenster Labels über die Knöpfe legt, finde ich es irreführend, dass ich das testen soll.
Commit 5b38946 habe ich schnell getestet, erstmal 2 Probleme:

  • dem Aufruf von _applyExperiment(exp) in experiments.py fehlt ein Argument (ignore_is_public) -- hab mich hier noch nicht genug eingelesen, um zu wissen was da rein soll
  • update_coord_info wird aufgerufen, wobei updateCoordInfo definiert wird (in gui.py)

zum Testen hab ich ignore_is_public auf 0 gesetzt, und für Punkt 2 den Namen geändert, scheint einwandfrei zu funktionieren.

@TanDro
Copy link
Contributor Author

TanDro commented Oct 2, 2018

Alles klar, angepasst und eingecheckt.

@TanDro TanDro closed this as completed Oct 2, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhalmen jhalmen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TanDro @jhalmen