Graph - Githubissues

mslehre / MAS

Maximum Acyclic Subgraph (MAS) - Multiple Sequence Alignment (MSA) Game

1 stars 0 forks source link

Graph #68

Closed sebastianbierb closed 5 years ago

sebastianbierb commented 5 years ago

Graph verbessert und aktualisiert.
State.cc und State.h angepasst an Graph
Coding-Style
Ordner Colorpalette geloescht (mit Absprache von Lucas)
Warnungen in Graphrenderer werden noch von Lucas abgearbeitet, weil Code befindet sich in der Bearbeitung
neue Funktion consistent(Edge& e, Edge& f) in State.h von Maurice hinzugefuegt

sebastianbierb commented 5 years ago

Lucas wird in einem eigenen Pull request Graphrenderer.cc fixen bzgl. der Empfehlungen in den Kommentaren

TimonKapischke commented 5 years ago

I fix State.cc, State.h and State_test.cc in a seperad pull request.

MarioStanke commented 5 years ago

Damit kein Missverstaendnis aufkommt, wenn er so lange offen bleibt, schliesse ich den Pull-Request. Timon wartet aber wohl auf die Aenderung der Graph-Klasse bevor er seine Aenderungen eincheckt.f Es waere gut, wenn Sie wenigstens einen Teil meiner Vorschlaege, der schnell zu machen ist, als Pull-Request stellen.

sebastianbierb commented 5 years ago

Damit kein Missverstaendnis aufkommt, wenn er so lange offen bleibt, schliesse ich den Pull-Request. Timon wartet aber wohl auf die Aenderung der Graph-Klasse bevor er seine Aenderungen eincheckt.f Es waere gut, wenn Sie wenigstens einen Teil meiner Vorschlaege, der schnell zu machen ist, als Pull-Request stellen.

Guten Tag Herr Stanke, alle von ihnen vorgeschlagenen Änderungen in Graph.h und Graph.cc wurden schon gestern im Pull request übernommen. Lucas hat bestätigt das er bezüglich Graphrenderer.cc bzw. der Vizualition übernimmt. Mit Absprache von Timon werden State.cc, state.h und State_test.cc übernommen, das bedeutet Timon muss nicht auf weitere Änderungen bezüglich der Klasse Graph warten.

MarioStanke commented 5 years ago

Guten Tag Herr Stanke, alle von ihnen vorgeschlagenen Änderungen in Graph.h und Graph.cc wurden schon gestern im Pull request übernommen.

Das habe ich in der Konversation zum Pull-Request auf GitHub nicht gesehen. Ich habe regelmaessig drauf geguckt und auf eine Nachricht von Ihnen gewartet, dass der Branch gemerged werden soll. Ich oeffne den Pull-Request jetzt neu und bearbeite ihn.