Parser für Inputdaten

schmittlauch commented

2017-06-28 00:08:11 +02:00

Owner

closes #3

Ich habe im package JobShopParser 2 Module geschrieben:

JobShopParser.jobhop1_parser parst eine Liste von Probleminstanzen (zB /inputdata/jobshop1.txt)
JobShopParser.jobhop1_parser parst einzelne Probleminstanzen (zB /inputdata/jobshop2/*)

Denkt dran, die neuen benötigten Python Pakete aus der requirements.txt zu installieren.

closes #3 Ich habe im package JobShopParser 2 Module geschrieben: - JobShopParser.jobhop1_parser parst eine Liste von Probleminstanzen (zB /inputdata/jobshop1.txt) - JobShopParser.jobhop1_parser parst einzelne Probleminstanzen (zB /inputdata/jobshop2/*) Denkt dran, die neuen benötigten Python Pakete aus der requirements.txt zu installieren.

schmittlauch commented

2017-06-28 00:11:30 +02:00

Author

Owner

meinetwegen kann ich auch noch nen kleinen Doctest schreiben, die Benutzung wird aber zumindest durch die Beispiel- main() Funktionen in den einzelnen Modulen deutlich.

Have fun

meinetwegen kann ich auch noch nen kleinen Doctest schreiben, die Benutzung wird aber zumindest durch die Beispiel- main() Funktionen in den einzelnen Modulen deutlich. Have fun

schmittlauch added the

enhancement

label 2017-06-28 00:19:53 +02:00

maxschlosser added this to the Necessary functionality milestone 2017-06-28 11:14:50 +02:00

maxschlosser self-assigned this 2017-06-28 19:21:14 +02:00

maxschlosser added the

wants review

label 2017-06-28 19:21:22 +02:00

maxschlosser commented

2017-06-28 20:09:10 +02:00

Owner

@schmittlauch Wäre es sehr aufwändig das JobShopProblem iterable zu machen? Bzw, dafür zu sorgen, dass wenn eine Funktion die ein iterable erwartet darauf aufgerufen wird einfach direkt die problem_data verwendet wird?

Wär nur ein kleiner Schönheitsfehler der mir aufgefallen ist, sonst scheint alles gut zu funktionieren.

Btw: Ich würde vorschlagen den JobShop bei allen Namen wegzulassen, das es zu dem Projekt gehört ist irgendwie klar. Dadurch würden die meisten Namen kürzer und lesbarer. Statt JobShopParser.jobshop1_parser.parse_jobshop1_file() zb Parser.js1_style.parse_file()

@schmittlauch Wäre es sehr aufwändig das `JobShopProblem` `iterable` zu machen? Bzw, dafür zu sorgen, dass wenn eine Funktion die ein `iterable` erwartet darauf aufgerufen wird einfach direkt die `problem_data` verwendet wird? Wär nur ein kleiner Schönheitsfehler der mir aufgefallen ist, sonst scheint alles gut zu funktionieren. Btw: Ich würde vorschlagen den JobShop bei allen Namen wegzulassen, das es zu dem Projekt gehört ist irgendwie klar. Dadurch würden die meisten Namen kürzer und lesbarer. Statt `JobShopParser.jobshop1_parser.parse_jobshop1_file()` zb `Parser.js1_style.parse_file()`

maxschlosser commented

2017-06-28 20:27:53 +02:00

Owner

Glaubst du es wär möglich die beiden Grammatiken in einen einzigen Parser zu packen (jobshop2 müsste ja in jobshop1 enthalten sein) oder ist das zu viel Aufwand?

Ist nicht notwendig, es funktioniert ja, wär nur quality of life.

Glaubst du es wär möglich die beiden Grammatiken in einen einzigen Parser zu packen (jobshop2 müsste ja in jobshop1 enthalten sein) oder ist das zu viel Aufwand? Ist nicht notwendig, es funktioniert ja, wär nur quality of life.

schmittlauch commented

2017-06-28 21:06:41 +02:00

Author

Owner

Glaubst du es wär möglich die beiden Grammatiken in einen einzigen Parser zu packen (jobshop2 müsste ja in jobshop1 enthalten sein) oder ist das zu viel Aufwand?

So dass der Parser selbstständig erkennt, welches Format vorliegt?
Die JS1 Grammatik beinhaltet zwar die js2, dennoch muss ich beim parsen die root node der Grammatik angeben.
Ne ganz simple Idee wäre, erst auf die eine Art zu parsen und wenn das failt auf die andere zu parsen.

Das mit dem Iterable kann ich machen, kümmer ich mich morgen drum. Und Namen sind doch eh Schall und Rauch...

> Glaubst du es wär möglich die beiden Grammatiken in einen einzigen Parser zu packen (jobshop2 müsste ja in jobshop1 enthalten sein) oder ist das zu viel Aufwand? So dass der Parser selbstständig erkennt, welches Format vorliegt? Die JS1 Grammatik beinhaltet zwar die js2, dennoch muss ich beim parsen die root node der Grammatik angeben. Ne ganz simple Idee wäre, erst auf die eine Art zu parsen und wenn das failt auf die andere zu parsen. Das mit dem Iterable kann ich machen, kümmer ich mich morgen drum. Und Namen sind doch eh Schall und Rauch...

schmittlauch commented

2017-06-28 21:35:39 +02:00

Author

Owner

Worüber soll das iterable denn iterieren bzw. was soll es jeweils zurückgeben? Jeweils einen der Jobs?