Vorläufige Korrektur

[Kegny]

Eure Abgabe ist schon sehr gut, aber so noch nicht ganz in Ordnung.
Ansonsten würde ich gerne noch von edsch103 und facho100 mehr sehen,
das bisherige ist deutlich zu wenig. Ihr könnt euch z.B. an die Bugs setzten oder
im Ilias auf die von Jens gemachten Vorschläge eingehen.
(Die meisten "Veränderungen" kommen durch gepushte binäre Dateien, halt die Kompilate)

Zu lösende Probleme:
SEHR WICHTIG:

• Gradle nicht verwendet; Travis läuft IMMER wegen der leeren gradlew-Datei
  WICHTIG:
• Schreibe-Test-Phasen können gewechselt werden trotz Syntax-Fehler
• Fehler-Ausgabe in der Konsole ist zwar schon einmal besser als gar nichts, aber
• Eine Fehlermeldung bei Syntax/Test-Fehlern in der GUI ist Notwendig, damit
  der Benutzer weiß, warum es nicht weitergeht

Wäre schön, wenn:

• Das Rädchen wiederspricht sich bei Fehlern mit der Konsole und dem Tatsächlichem Modus

Ansonsten ist das Tracking sehr gelungen und die Babysteps funktionieren.
(Ganz abgesehen von der schönen Oberfläche)

Hinweise:
Ihr hättet eine .gitignore anlegen und pushen können/sollen, in der Ihr generierte Dateien,
wie .class-Dateien filtert. Die gehören nicht mit git versioniert. Worauf ihr auch noch achten
solltet ist keine ungelösten Konflikte zu pushen, das heißt, dass ihr die lösen müsst und dann
den Commit verfassen und nicht einfach noch einmal von vorne probieren den Commit zu verfassen.

[sonjastange]

Nur um Missverständnissen vorzubeugen: Bedeutet das, dass Fahim und Eduard diese Änderungen machen sollen und Yordan und ich quasi bereits "fertig" sind?

[facho100]

Hi,

Ich bin froh, dass unsere Arbeit halbwegs ok ist. Möchte nun dir nicht zu nahe treten aber, ich wollte gerne wissen wie diese Auswertung zustande gekommen ist? Werden nur die commits aus den letzten zwei Wochen angeschaut?

Edsch103 und ich (facho100) haben nämlich die ganzen Anfangsklassen also praktisch das Grundgerüst des Programms (Loader, GUI z.B) geschrieben, welche wir (die Gruppe) über die Bearbeitungszeit in verschieden weiteren Klassen verpackt haben. Selbstverständlich auch andere Sachen, die commits sollten ausreichend Einsicht in dies gewähren. Im Nachhinein habe ich erfahren, dass Jens in einer Vorlesung vermerkt hat, dass wir die commits mit unseren Namen betiteln sollten (falls wir zu zweit daran gearbeitet haben), dies haben wir leider nicht berücksichtigt. Ich weiß leider auch nicht wie viel Einblick man in die einzelnen commits bekommt.

Generell haben wir sehr viel während unsere Arbeitsphase kommuniziert, Erweiterungen und Aussehen der GUI beispielsweise liefen nur nach Absprache, da diese jedoch nicht dokumentiert sind kann ich nachvollziehen dass man da keine Rücksicht nehmen kann.

Wie Sonja nun in einer anderen Email bereits gefragt hat, sind die beiden anderen „fertig“ mit dem Projekt?

Wir werden nachwievor versuchen die Probleme, die du geschildert hast zu lösen, wollte nur nicht dass unsere Arbeit umsonst gewesen ist oder es so rüber kommt dass wir nichts gemacht haben.

Grüße Fahim

Von: Kegny [mailto:[email protected]]
Gesendet: Dienstag, 19. Juli 2016 17:33
An: ProPra16/ProPra16Team1 [email protected]
Betreff: [ProPra16/ProPra16Team1] Vorläufige Korrektur (#1)

Eure Abgabe ist schon sehr gut, aber so noch nicht ganz in Ordnung.
Ansonsten würde ich gerne noch von edsch103 und facho100 mehr sehen,
das bisherige ist deutlich zu wenig. Ihr könnt euch z.B. an die Bugs setzten oder
im Ilias auf die von Jens gemachten Vorschläge eingehen.
(Die meisten "Veränderungen" kommen durch gepushte binäre Dateien, halt die Kompilate)

Zu lösende Probleme:
SEHR WICHTIG:

• Gradle nicht verwendet; Travis läuft IMMER wegen der leeren gradlew-Datei WICHTIG:
• Schreibe-Test-Phasen können gewechselt werden trotz Syntax-Fehler
• Fehler-Ausgabe in der Konsole ist zwar schon einmal besser als gar nichts, aber
• Eine Fehlermeldung bei Syntax/Test-Fehlern in der GUI ist Notwendig, damit der Benutzer weiß, warum es nicht weitergeht

Wäre schön, wenn:

• Das Rädchen wiederspricht sich bei Fehlern mit der Konsole und dem Tatsächlichem Modus

Ansonsten ist das Tracking sehr gelungen und die Babysteps funktionieren.
(Ganz abgesehen von der schönen Oberfläche)

Hinweise:
Ihr hättet eine .gitignore anlegen und pushen können/sollen, in der Ihr generierte Dateien,
wie .class-Dateien filtert. Die gehören nicht mit git versioniert. Worauf ihr auch noch achten
solltet ist keine ungelösten Konflikte zu pushen, das heißt, dass ihr die lösen müsst und dann
den Commit verfassen und nicht einfach noch einmal von vorne probieren den Commit zu verfassen.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, #1 view it on GitHub, or https://github.com/notifications/unsubscribe-auth/ATF0V_U9mL-vJML6D-ngJrjIHAmizno3ks5qXO4pgaJpZM4JP23N mute the thread.

[Kegny]

@Sonja, wobei auch für den Rest interessant:
Nein, die Änderungen sollen irgendwie unter euch aufgeteilt werden (Gruppenprojekt und ich forder ja nur Teile der Aufgabe nach).
Natürlich könnt ihr das den beiden aufdrücken, damit die auch noch ein wenig zusätzlichen Beitrag bekommen,
damit das für die ausreicht. Oder die machen halt noch ein Zusatzfeature, wie oben beschrieben.
"Quasi fertig" möchte ich das auch nicht nennen, einfach damit Ihr euch nicht zurücklehnt, da es nicht mehr euer Problem ist; Das Programm erfüllt nun einmal noch nicht
ausreichend die Aufgabenstellung

@fahim, wobei auch für den Rest interessant:
Vorne weg, ich nehme dir das nicht übel, kann ja sein, dass ich nen Fehler gemacht habe

Und nein, es werden alle Commits angeschaut, nicht nur die am Ende, wenn Du es ganz genau wissen willst:
Habe ich mir zuerst im Graph eine Übersicht und Ersteindruck geholt, wer wie viele Zeilen erzeugt hat;
Danach für jeden einzeln bin ich dann durch den Großteil der Commits und habe festgestellt,
wessen Graph künstlich durch binäre Dateien, oder triviale Commits, wie tonnenweise leerzeilen
(Kam tatsächlich bei einem Team vor), hoch ist.
Dann hätte ich noch mir die Issues angesehen, aber die habt ihr nicht verwendet.(Ist in Ordnung)
Mehr hätte ich mir eigentlich nur ansehen können, wenn ich mir alle Commits angesehen hätte, oder vergesse ich da was?

Und Ihr braucht wirklich nicht so viel Angst zu haben, "deutlich zu wenig" schrieb ich nur, um auch wirklich dann zur Arbeit zu bewegen. (Wäre bitter beim letzten Projekt noch rauszufliegen, oder?)
Bei euch ist die Aufteilung relativ ausgeglichen, weswegen nicht SO viel zu tun ist
und die Bugfixe eigtl ausreichen SOLLTEN.

Ich hoffe, das beantwortet soweit eure Fragen? Ansonsten könnt Ihr hier gerne weitere stellen.

[edsch103]

Hallo,
habe jetzt an einigen Bug-Fixes gearbeitet und das richtige gradle-build erstellt.
Hoffe das dies für mich ausreicht und für Fahim wenn er sich darum kümmert die jetzigen Fehlermeldungen die im Moment nur Konsolenausgaben sind als Label darzustellen, so dass die jetzigen Kritikpunkte am Projekt bearbeitet wären.

Mit freundlichen Grüßen.

[Kegny]

Hallo,
ja, vom Umfang her sieht das bei dir schon einmal gut aus (zum Testen komme ich erst am Anfang des WE wieder)

Viele Grüße

[Kegny]

Aktueller Stand ``b5702befa20d65ed6bf894317be2937709369a6c`:
public static String exc_file musste Ich selbst setzten, da Ihr sonst euren Aufgabenkatalog.txt nicht findet, da stimmt der Pfad nicht

Außerdem bin ich inzwischen dazu gekommen mir deine letzten Commits anzusehen und testen, @edsch103, die gradlew hinzuzufügen, zu löschen und wieder hinzuzufügen ist nicht ausreichend.

Genauso, wie die .gitignore alleine nicht ausreichend ist @facho100

[edsch103]

Hallo,
der Fehler mit dem Aufgabenkatalog tritt bei mir jetzt nicht ein, insbesondere da das ganze ähnlich wie curTest curClass behandelt wird wäre es gut das bitte etwas näher zu spezifizieren, damit ich mich darum kümmern kann.
Ich verstehe jetzt aber ansonsten nicht ganz warum die Leistung nicht ausreichend ist, immerhin wurden neben den von mir aus unnötigen Commits , die eigentlich für mich eher als Rückmeldung gedacht war das travis nichts leeres kompiliert, die oben genannten bugs doch gefixed + die Labeldarstellung angenommen.

Mit freundlichen Grüßen

[Sonja]

This message was created automatically by mail delivery software.

A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:

[email protected]
Domain kisa.ca has exceeded the max emails per hour (1/1 (100%)) allowed. Message will be reattempted later

------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext2.iad.github.net ([192.30.252.193]:40929 helo=github-smtp2b-ext-cp1-prd.iad.github.net)
by vps.depositoaprazo.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.87)
(envelope-from [email protected])
id 1bRO6O-00052l-M4
for [email protected]; Sun, 24 Jul 2016 19:26:13 +0100
Date: Sun, 24 Jul 2016 11:25:31 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1469384731;
bh=XkH7vlVcOWq+cxCuedQWpt3IvHella2uKeXbWXyvt00=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=IUN41V4K04eUGP2f0wqQx1x/AFZmWoXVuGPO4d0CoEWD+eAkoHlobuEErlfes7Mau
+D6Rr736oaLzAi283sWxWHBrSrp2RCsjQFIiHSj+0pfg8zv3eEYpQwNlNxnV0uV7Y8
dGoFxw2hEAvqtovJ0R0+AA8j4Hm2ZeOZmObGmb/E=
From: edsch103 [email protected]
Reply-To: ProPra16/ProPra16Team1 [email protected]
To: ProPra16/ProPra16Team1 [email protected]
Cc: Sonja Lang [email protected],
Mention [email protected]
Message-ID: ProPra16/ProPra16Team1/issues/1/[email protected]
In-Reply-To: ProPra16/ProPra16Team1/issues/[email protected]
References: ProPra16/ProPra16Team1/issues/[email protected]
Subject: Re: [ProPra16/ProPra16Team1] Vorläufige Korrektur (#1)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5795081af3c70_798d3fda5f3c72a010828e1";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: edsch103
X-GitHub-Recipient: Sonja
X-GitHub-Reason: mention
List-ID: ProPra16/ProPra16Team1 <ProPra16Team1.ProPra16.github.com>
List-Archive: https://github.com/ProPra16/ProPra16Team1
List-Post: mailto:[email protected]
List-Unsubscribe: mailto:unsub+00021136b5583e5e05faa20224e9086305d057e06777be8092cf0000000113acca1a92a169ce09ea7ceb@reply.github.com,
https://github.com/notifications/unsubscribe/AAIRNt_WwjhhJ0bskNEyJzwFRufeRGeVks5qY64agaJpZM4JP23N
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address: [email protected]

----==_mimepart_5795081af3c70_798d3fda5f3c72a010828e1
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hallo,
der Fehler mit dem Aufgabenkatalog tritt bei mir jetzt nicht ein, insbeso=
ndere da das ganze =C3=A4hnlich wie curTest curClass behandelt wird w=C3=A4=
re es gut das bitte etwas n=C3=A4her zu spezifizieren, damit ich mich dar=
um k=C3=BCmmern kann.
Ich verstehe jetzt aber ansonsten nicht ganz warum die Leistung nicht aus=
reichend ist, immerhin wurden neben den von mir aus unn=C3=B6tigen Commit=
s , die eigentlich f=C3=BCr mich eher als R=C3=BCckmeldung gedacht war da=
s travis nichts leeres kompiliert, die oben genannten bugs doch gefixed +=
die Labeldarstellung angenommen.

Mit freundlichen Gr=C3=BC=C3=9Fen

---

You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
#1 (comment)

----==_mimepart_5795081af3c70_798d3fda5f3c72a010828e1
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable

Hallo,
der Fehler mit dem Aufgabenkatalog tritt bei mir jetzt nicht ein, insbeso= ndere da das ganze =C3=A4hnlich wie curTest curClass behandelt wird w=C3=A4= re es gut das bitte etwas n=C3=A4her zu spezifizieren, damit ich mich dar= um k=C3=BCmmern kann.
Ich verstehe jetzt aber ansonsten nicht ganz warum die Leistung nicht aus= reichend ist, immerhin wurden neben den von mir aus unn=C3=B6tigen Commit= s , die eigentlich f=C3=BCr mich eher als R=C3=BCckmeldung gedacht war da= s travis nichts leeres kompiliert, die oben genannten bugs doch gefixed += die Labeldarstellung angenommen.

Mit freundlichen Gr=C3=BC=C3=9Fen

&m= dash;
You are receiving this because you were mentioned.
Reply = to this email directly, view it on GitHub, or mute the thread.

<meta itemprop=3D"description" content=3D"View this Issue on GitHub"></me=
ta>

----==_mimepart_5795081af3c70_798d3fda5f3c72a010828e1--

[edsch103]

Hallo,
habe jetzt nochmal eine Art Pair-Programming Feature eingebaut, bei dem jeweils ein User für die Tests und der andere für die Klasse mit dem Refactoren zuständig ist. Außerdem hab ich den .txt Namen von Aufgabenkatalog weggelassen weil ich mir den Fehler sonst nicht erklären kann, da dieser wie gesagt bei mir nicht aufrtitt.

MFG

[facho100]

Hallo,

Neben der gitignore habe ich noch bei der Darstellung der Fehlermeldung beigetragen, was auch bei meinem eigenen commit zu sehen ist in Form eines Labels und sonst noch bei Eduard's commit. Damit ist doch alles von der liste abgehackt?

EDIT: Was genau sollen wir bzw. ich noch machen damit wir sicher sind? Weil das Programm läuft ja so schon einwandfrei.

MfG

[Kegny]

Hallo,

• Euer Projekt buildet immer noch nicht mit gradle: gradle build buildet eine quasi leere jar-Datei und gradle check führt eure Unit-Tests nicht aus (kann man einfach testen, indem man einen fehlschlagenden Test hinzufügt).
• Ihr habt außerdem ein Package de.hhu.TDD, was wohl nicht mehr inkludiert sein sollte. (?)

Mit @edsch103 wäre ich jetzt zufrieden, @facho100 sollte diese 2 Punkte bearbeiten, dann wäre das auch i.O.

Viele Grüße

[Sonja]

This message was created automatically by mail delivery software.

A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:

Domain  has exceeded the max emails per hour (1/1 (100%)) allowed.  Message will be reattempted later

[facho100]

Hi,

habe das Package de.hhu.TDD mal gelöscht. Mussten das einem Punkt reinpacken weil sonst das Programm nicht funktioniert hätte. So als Backup.

Wegen Gradle habe ich den Path der Klassen verändert. So wie ich gelesen haben braucht gradle die files in einem bestimmten Path daher habe ich einige Klassen so wie die Resource files verschoben. Programm läuft normal, weiß nur nicht ob das Gradle jetzt ordnungsgemäß funktioniert und wie ich das selber kontrollieren kann. Hoffe du kannst da fix rüber schauen.

MfG Fahim

[facho100]

Hey Kegny,

das build funktioniert hoffe ich jetzt (da es auf Travis jetzt auf passed). Falls es noch Probleme geben soll diesbezüglich würde ich mich freuen wenn du dich noch vor der Frist melden würdest.

MfG Fahim

EDIT: Zwischenzeitig hat es nicht funktioniert deswegen habe ich hier nochmal einem Nachricht geschrieben