These are chat archives for pichuio/halftone

10th
Jun 2015
cproduct
@cproduct
Jun 10 2015 01:20
ist drin... habs als command line argument rein gemacht zum aktivieren: -threadType 1
cproduct
@cproduct
Jun 10 2015 01:28
ganz wichtig jetzt: in der doku einen neuen kapitel rein machen, der den algorithmus beschreibt zusätzlich zur anderen implementation.
und mal testen, ob das bei nem ganz kleinen bild so funktioniert
cproduct
@cproduct
Jun 10 2015 09:00
Bei Bildern kleiner 9 breite gehts nicht..genauso wie 19
Oder sonst was
Patrick D.
@pichuio
Jun 10 2015 09:06
hmm
wie war prüfung?
cproduct
@cproduct
Jun 10 2015 09:07
Gut
Fixst du die Kacke und machst mal fertig? Nervt mich langsam alles.
Patrick D.
@pichuio
Jun 10 2015 09:09
bin schon am doku schreiben
cproduct
@cproduct
Jun 10 2015 09:09
Ja und code dokuementerien und plapla
Da hast ewig Zeit und musst es dann kurz vor Abgabe machen und bei allem brauchst Hilfe
Patrick D.
@pichuio
Jun 10 2015 09:13
ja sorry.....mich nervts doch selber
kannst du mal kurz in die Doku schauen ob die Abbildung stimmt
bzw die Tabelle
cproduct
@cproduct
Jun 10 2015 09:14
Nachher kann ich des schon
Machst du auch so n Diagramm für die Zeitmessung wie neemann?
Patrick D.
@pichuio
Jun 10 2015 09:15
ja
cproduct
@cproduct
Jun 10 2015 09:15
Haja gut. Da fehlt überall auch noch n weng Text glaub...
Patrick D.
@pichuio
Jun 10 2015 09:21
ja...wollte nur wissen ob die Tabelle stimmt
in welchem Dokument ist den das Diagramm mit der Zeitmessung...nicht in Moodle..oder?
finde das gerade nicht
Patrick D.
@pichuio
Jun 10 2015 10:17
ok..habs
Patrick D.
@pichuio
Jun 10 2015 10:30
laufen bei dir alles Tests durch?
bin jetzt kurz an der DH
cproduct
@cproduct
Jun 10 2015 10:32
die test für den autoresetevent manchmal nicht
die beschreibung in der doku ist falsch
für den ditherthread2
cproduct
@cproduct
Jun 10 2015 10:38
da bearbeitet immer noch jeder thread die komplette zeile
und gibt der darunter liegen zeile bzw. thread bescheid (mit nem Ereignis), wann er mit welcher sektion beginnen kann
cproduct
@cproduct
Jun 10 2015 10:53

und bei dem cobertura funktionieren zwei andere test fälle nicht
aber mit mvn install laufen se erfolgreich durch
Patrick D.
@pichuio
Jun 10 2015 12:32
hab die doku korrigiert
ich hab in meinen Per
persönlichen Bericht reingeschrieben dass zu die Java implementierung quasi vollständig gemacht hast...nur zur Info
hab dann noch was zu der parallelen Architektur in Go geschrieben, die die ich gemacht hab...allerdings das mit dem sync nicht funktioniert hat
cproduct
@cproduct
Jun 10 2015 12:44
ich schmier noch bissl in der doku rum jetzt
des diagramm machen wir einfach mit den libre office diagramm ding oder?
Patrick D.
@pichuio
Jun 10 2015 12:45
ja
cproduct
@cproduct
Jun 10 2015 12:48
machst du gschwind n paar tests mit 3 verschieden großen bildern und mit 1 2 4 threads?
Patrick D.
@pichuio
Jun 10 2015 12:57
jep
die zeiten notier ich dann kurz in der Tabelle
cproduct
@cproduct
Jun 10 2015 13:05
habs eingecheckt
nimmst halt "gute" zeiten :D
ach noch ne kleinigkeit gefunden
paar minuten
done
cproduct
@cproduct
Jun 10 2015 13:12
so wie ich das sehe ist jetzt noch offen: quellcode vollständig dokumentieren; doku: performance + startparameter beispiel (java -jar gruppe17.jar -imagePath "XXX" ..."); cobertura test cases fixen
Patrick D.
@pichuio
Jun 10 2015 13:18
in unserem drive Ordner ist das Dokument mit den Zeiten das ich gerade am füllen bin
aktuell bin ich noch bei variante 1 der DitherThreads..variante 2 kommt anschließen
cproduct
@cproduct
Jun 10 2015 13:19
okay gut
ach noch offen ist da smit den ganz kleinen bildern
machst du die messungen aber nicht über die ide oder?
Patrick D.
@pichuio
Jun 10 2015 13:28
mach gerade beides...zahlen sind aber aus der commando zeile
cproduct
@cproduct
Jun 10 2015 13:29
ok passt. weil die ide verlangsamt des normal
Patrick D.
@pichuio
Jun 10 2015 13:30
jo...gehts bei dir schneller bei dem sample_.jpg
cproduct
@cproduct
Jun 10 2015 13:31
zwischen ide und commandozeile?
Patrick D.
@pichuio
Jun 10 2015 13:31
ne..verglichen zu meinen zahlen
cproduct
@cproduct
Jun 10 2015 13:32
moment
gleich
Patrick D.
@pichuio
Jun 10 2015 13:35
kein stress
cproduct
@cproduct
Jun 10 2015 13:35
die zeiten sind gleich :D
Patrick D.
@pichuio
Jun 10 2015 13:35
ok
meine testbilder lad ich auch noch mit hoch...oder
cproduct
@cproduct
Jun 10 2015 13:36
haja schon
für die doku sind halt echt die abmessungen interessant und dein prozessor
Patrick D.
@pichuio
Jun 10 2015 13:39
ja...das schreib ich dazu
cproduct
@cproduct
Jun 10 2015 13:44
alta dieser cobertura. ich dreh durch
Patrick D.
@pichuio
Jun 10 2015 14:03
so...Tabelle ist fertig
cproduct
@cproduct
Jun 10 2015 14:04
top
cobertura läuft auch wieder durch
Patrick D.
@pichuio
Jun 10 2015 14:06
top
wenn ich -threadType 1 sag, wird DitherThread2 genommen, oder?
cproduct
@cproduct
Jun 10 2015 14:07
ja genau
Patrick D.
@pichuio
Jun 10 2015 14:07
gut
cproduct
@cproduct
Jun 10 2015 14:07
da müssten bessere zeiten raus kommen
Patrick D.
@pichuio
Jun 10 2015 14:08
mal ja..mal nein
:worried:
cproduct
@cproduct
Jun 10 2015 14:08
auch wurst
bei großen bildern normal schon
ich weiß da auch was das problem ist aber ich hab kein bock mehr
Patrick D.
@pichuio
Jun 10 2015 14:08
ich glaub dass das eventuell durch das benchmark script verfälscht wird
ist halt dann unter volllast der rechner
cproduct
@cproduct
Jun 10 2015 14:09
das muss er ja
aber auch nur für n paar millisekunden
Patrick D.
@pichuio
Jun 10 2015 14:09
ja
soll ich die Tabelle als pdf in das Repo mit reinpacken?
oder exel datei
cproduct
@cproduct
Jun 10 2015 14:10
ich würde das gar nicht rein machen sondern nur in die doku n diagramm
Patrick D.
@pichuio
Jun 10 2015 14:13
ok...dann genereier ich n paar diagramme aus der Tabelle
cproduct
@cproduct
Jun 10 2015 14:13
2?
Patrick D.
@pichuio
Jun 10 2015 14:14
ist mega unübersichtlich mit den ganzen werten...oder?
cproduct
@cproduct
Jun 10 2015 14:14
ja hast im neemann seins nicht gesehen (hat er 2 mal gezeigt)
Patrick D.
@pichuio
Jun 10 2015 14:14
dann mach ich so
cproduct
@cproduct
Jun 10 2015 14:15
des ding hat der glaub nirgends hochgeladen
einfach eine sequenzielle zeit für jedes image und dann pro thread anzahl eine
und dann 1 diagramm ditherthread1 und 1 diagramm für ditherthread 2
Patrick D.
@pichuio
Jun 10 2015 14:16
ja...so mach ich s jetzt gerade
cproduct
@cproduct
Jun 10 2015 14:16
und da nimmst halt für uns gute zeiten
Patrick D.
@pichuio
Jun 10 2015 14:16
jep
also parallel gute und seq schlechte
cproduct
@cproduct
Jun 10 2015 14:19
genau
bei dem sample_.jpg kann doch bei dem ditherthread was nicht stimmen
die zeiten für des sequenzielle sind viel zu schnell
die tests kannst nochmal machen
du hast vergessen "mvn install"
10/06/2015 15:44:38.371 : ALL : StartupConfig [ditherMode=Both, matrixType=Type2x3, images=[org/sample_.jpg], parallelConfig=ParallelConfig [threadCount=2]saveFilesfalse]
des ist die alte meldung von gestern oder vorgestern
Patrick D.
@pichuio
Jun 10 2015 14:24
verdammt...
stimmt das wollte ich vorhin machen ist dann aber fehlgeschlagen
und ich habs vergessen neu zu machen
cproduct
@cproduct
Jun 10 2015 14:24
jo
Patrick D.
@pichuio
Jun 10 2015 14:24
...ich machs kutz
kurz
cproduct
@cproduct
Jun 10 2015 14:25
...
Patrick D.
@pichuio
Jun 10 2015 14:25
wenns install gehen würde XD
cproduct
@cproduct
Jun 10 2015 14:25
ich lösch mal die daten von den tests weg ausm github
Patrick D.
@pichuio
Jun 10 2015 14:25
jo ok
aber nciht mein benchmark script XD
cproduct
@cproduct
Jun 10 2015 14:26
haha
klar
welcher testfall macht da probleme?
da musst dann halt danach schauen
Patrick D.
@pichuio
Jun 10 2015 14:27
der imagehelpertest
cproduct
@cproduct
Jun 10 2015 14:27
ja dann musst mal fixen :D
Patrick D.
@pichuio
Jun 10 2015 14:27
jo...schon dabei
cproduct
@cproduct
Jun 10 2015 14:28
mein repo ist vermurkst
du hast da irgendwas hochgeladen was meinem system nicht gefällt
"Y:\THIS\PATH\SHOULD\NOT\EXIST.jpg" hast du hochgeladen
löscht du die mal raus und machst n commit
Patrick D.
@pichuio
Jun 10 2015 14:36
ja...der pfad gefällt meinem system nich..das war das problem
cproduct
@cproduct
Jun 10 2015 14:36
ich weiß
hat auf git trotzdem nix verloren
Patrick D.
@pichuio
Jun 10 2015 14:37
ich find die datei gar nich
cproduct
@cproduct
Jun 10 2015 14:41
ach egal. hats bei mir angezeigt. jetzt kann ich halt nix mehr hochladen
Patrick D.
@pichuio
Jun 10 2015 14:41
ne...die muss ja iwo sein
cproduct
@cproduct
Jun 10 2015 14:42
wenn man sich die details von dem commit ansieht ist die drin
aber danach irgendwie nicht mehr
Patrick D.
@pichuio
Jun 10 2015 14:44
bei mir gibts die nicht
is verschwunden
cproduct
@cproduct
Jun 10 2015 14:45
egal
Patrick D.
@pichuio
Jun 10 2015 14:47
ich lass den Test jetzt mal so...hab in nur kurz bei mir geändert
ich weiß gar nicht wie man sowas für ubuntu etc testen kann...die legen immer alles an...iwo
cproduct
@cproduct
Jun 10 2015 14:48
ha n unzulässiger pfad halt angeben
lass einfach des Y:\ weg
Patrick D.
@pichuio
Jun 10 2015 14:49
wird trotzdem angelegt
cproduct
@cproduct
Jun 10 2015 14:49
geiler shit
dann lösch den test fall raus
hast neue zeiten?
Patrick D.
@pichuio
Jun 10 2015 14:51
läuft gerade durch
deutlich schneller :D
also version ditherthread2
cproduct
@cproduct
Jun 10 2015 14:55
logisch :D
bei deinen tests von oben hat er den ditherthread 2 gar nicht benutzt wahrscheinlich
Patrick D.
@pichuio
Jun 10 2015 14:56
wahrscheinlich
cproduct
@cproduct
Jun 10 2015 14:57
bei dem ditherthread1 müssten die zeiten etwa gleich sein, egal wie viele threads oder?
also sequenziell = parallel (mit n bissl schneller)
Patrick D.
@pichuio
Jun 10 2015 14:57
muss erst warten bis ditherthread2 durchgelaufen ist
cproduct
@cproduct
Jun 10 2015 14:58
ok
Patrick D.
@pichuio
Jun 10 2015 15:11
also fast doppelt so schnell ditherthread2
mit 4 kernen
cproduct
@cproduct
Jun 10 2015 15:15
ja passt
ist zwar nicht top aber passt
Patrick D.
@pichuio
Jun 10 2015 15:19
so zeiten sind drin
den test hab ich rausgemacht
cproduct
@cproduct
Jun 10 2015 15:28
ok
des diagramm kannst ja direkt im writer machen
Patrick D.
@pichuio
Jun 10 2015 15:35
ja...wird wohl besser sein...weil des google ding is da schrott
XD
cproduct
@cproduct
Jun 10 2015 15:36
ja
Patrick D.
@pichuio
Jun 10 2015 15:43
soll ich wirklich alle 4 bilder in eine Tabelle?
dann wäre ein Spaltendiagramm aber besser
blob
so wäre das für alle 4 Bilder für DitherThread1
cproduct
@cproduct
Jun 10 2015 15:56
2 seq und 4 seq können ja weg
des ändert ja nix
einfach nur "seq" nennen
und dann links noch einheit
und die bezeichnungen für parallel noch anpassen, z.B. par (1 thread)
sonst blickt mans nicht auf anhieb
und bestimmt kann man da auch noch die werte anzeigen lassen ins diagramm
vllt auch noch direkt unter die bildernamen die pixelgröße
dann wäre das perfekt
Patrick D.
@pichuio
Jun 10 2015 16:11
so...jetzt kannst mal checken
pixelgröße ging leider nicht drunter
cproduct
@cproduct
Jun 10 2015 16:29
ich guck mal drüber und verbesser was mir auffällt
habs hochgeladen
eig. sollte man die zeiten fürs sequenzielle bei beiden diagrammen gleich haben
cproduct
@cproduct
Jun 10 2015 16:34
und eig. muss der parallele algorithmus mit einem thread immer n weng langsamer sein
Patrick D.
@pichuio
Jun 10 2015 16:38
dann mach ich das mal noch schnell
cproduct
@cproduct
Jun 10 2015 16:42
wäre gut
Patrick D.
@pichuio
Jun 10 2015 16:47
so ist drin
den jenkins hauts beim bauen immer raus :worried:
nicht genug ram
XD

´Java HotSpot(TM) 64-Bit Server VM warning: INFO: os::commit_memory(0x00000000fee29000, 196608, 0) failed; error='Cannot allocate memory' (errno=12)
#

There is insufficient memory for the Java Runtime Environment to continue.

Native memory allocation (mmap) failed to map 196608 bytes for committing reserved memory.

An error report file with more information is saved as:

/var/jenkins_home/jobs/halftone/workspace/hs_err_pid23236.log

ERROR: Maven JVM terminated unexpectedly with exit code 137
Finished: FAILURE´

haha XD
build läuft durch...tests nicht
cproduct
@cproduct
Jun 10 2015 16:52
ok
wird man ja wohl rausfinden
Patrick D.
@pichuio
Jun 10 2015 16:52
ja...der server hat zu wenig speicher
cproduct
@cproduct
Jun 10 2015 16:52
und seit wann gehts nicht mehr?
Patrick D.
@pichuio
Jun 10 2015 16:53
heute morgen...habe gestern morgen das letzte mal gebaut
cproduct
@cproduct
Jun 10 2015 16:53
dann wird man ja wohl den grund finden?!
Patrick D.
@pichuio
Jun 10 2015 16:53
ja das liegt nicht an unserem projekt
da passt alles
cproduct
@cproduct
Jun 10 2015 16:53
das glaube ich nicht
sonst hätte es nie funktioniert
da ist ja was rein gekommen was dazu führt
Patrick D.
@pichuio
Jun 10 2015 16:54
wenn ich das gerade richtig sehe ist der speicher vom jenkins voll
also die Festplatte
cproduct
@cproduct
Jun 10 2015 16:54
des ist eig. sowieso dumm des halt feature zu nennen. da gabs ja bei jedem lauf irgendein problem
Patrick D.
@pichuio
Jun 10 2015 16:55
ja dann nehmen wir es einfach raus
cproduct
@cproduct
Jun 10 2015 16:55
sozusagen das tool nicht benutzt
Patrick D.
@pichuio
Jun 10 2015 16:57
ich nehms aus der doku raus
ist vielleicht besser
cproduct
@cproduct
Jun 10 2015 16:57
ja
du weißt ja was dann noch zu tun ist, damits projekt fertig ist
Patrick D.
@pichuio
Jun 10 2015 16:59
das mit den kleinen Bildern...und was noch?
cproduct
@cproduct
Jun 10 2015 16:59
quelltext dokumentieren
?
Patrick D.
@pichuio
Jun 10 2015 17:00
stimmt
cproduct
@cproduct
Jun 10 2015 17:00
die startparameter in der doku noch erklären mit beispiel
in die doku noch n paar sätze
bei dem performance deine testumgebung
Patrick D.
@pichuio
Jun 10 2015 17:01
...also ob der so kleine Bilder da reinschmeißt
:D
cproduct
@cproduct
Jun 10 2015 17:01
da reichts nur eine zahl im testfall anzupassen
mit cobertura schauen wie viel testabdeckung --> und vllt mal selber nen test schreiben?
cproduct
@cproduct
Jun 10 2015 17:07
und dann die testabdeckung noch in die doku. da steht grad noch ein XXX
und mal nach TODO suchen (z.b. // TODO Auto-generated catch block)
Patrick D.
@pichuio
Jun 10 2015 18:25
die getter und setter kommentier ich aber nicht
cproduct
@cproduct
Jun 10 2015 18:25
jo passt
Patrick D.
@pichuio
Jun 10 2015 18:26
sourc doku müssten glaub jetzt passen
wenn ich nix vergessen hab
cproduct
@cproduct
Jun 10 2015 18:27
ach des passt schon so
hast des mal als html doku angeschaut?
Patrick D.
@pichuio
Jun 10 2015 18:27
ja
sieht ganz ordentlich aus
cproduct
@cproduct
Jun 10 2015 18:27
top
Patrick D.
@pichuio
Jun 10 2015 18:27
außer dass wenn es generiert wird halt fast 50 errors anzeigt
das stört mich n bissl
cproduct
@cproduct
Jun 10 2015 18:27
was sind des für welche?
Patrick D.
@pichuio
Jun 10 2015 18:28
aber das will auch das getter und setter kommentiert sind
cproduct
@cproduct
Jun 10 2015 18:28
haha
dann kannst des ja noch als fleißarbeit machen. wenns nur 50 sind gehts ja
10 minuten
Patrick D.
@pichuio
Jun 10 2015 18:29
haja dann mach ichs noch
/**
 * 
 * @param l1
 * @param l2
 * @param i
 * @param j
 * @param in
 * @param out
 */
was war denn da was?
cproduct
@cproduct
Jun 10 2015 18:32
des siehst doch im code
Patrick D.
@pichuio
Jun 10 2015 18:32
schon...aber dass ich da auch die richtigen bezeichnung nehm
cproduct
@cproduct
Jun 10 2015 18:32
l1 obere zeile
l2 untere zeile
Patrick D.
@pichuio
Jun 10 2015 18:32
l2 untere
cproduct
@cproduct
Jun 10 2015 18:32
i zeile
j spalte
Patrick D.
@pichuio
Jun 10 2015 18:33
jo
cproduct
@cproduct
Jun 10 2015 18:33
in -> in
Patrick D.
@pichuio
Jun 10 2015 18:33
hab ich
cproduct
@cproduct
Jun 10 2015 18:33
out -> out :D
Patrick D.
@pichuio
Jun 10 2015 18:33
ok
dann passts
so
Patrick D.
@pichuio
Jun 10 2015 18:38
source doku müsste passen
esse jetzt mal kurz was
cproduct
@cproduct
Jun 10 2015 18:40
sieht gut aus. hab bei mir auch mal die javadoc erstellt
!
des kann man so lassen
Patrick D.
@pichuio
Jun 10 2015 19:01
ok
cproduct
@cproduct
Jun 10 2015 19:29
was hast in data warehouse eig. für ne note?
Patrick D.
@pichuio
Jun 10 2015 19:34
2,0
cproduct
@cproduct
Jun 10 2015 19:34
haja
Patrick D.
@pichuio
Jun 10 2015 19:34
die haben paar blöden fragen gestellt...eigentlich architekturfragen
was hast du?
cproduct
@cproduct
Jun 10 2015 19:35
die haben nur dusslige fragen gestellt
1,0
:D
Patrick D.
@pichuio
Jun 10 2015 19:35
sauber XD
cproduct
@cproduct
Jun 10 2015 19:35
bei uns architekten haben 2 ne 1,0 und die anderen 2 ne 1,1
Patrick D.
@pichuio
Jun 10 2015 19:36
das hat er gesagt dass der schlechteste ne 1.1 hat :D
cproduct
@cproduct
Jun 10 2015 19:36
haha. sozusagen hast du wohl auch nur 10 punkte weniger
und die hab ich halt letztes semester mehr
Patrick D.
@pichuio
Jun 10 2015 19:36
jep...was hast du vorhin gemeint was noch in den tests anpassen zu ist?
cproduct
@cproduct
Jun 10 2015 19:37
ha wirklich gut sind die nicht
MainProcess ist gar nicht getestet
die start parameter sind auch nicht wirklich getestet
Patrick D.
@pichuio
Jun 10 2015 19:37
ok
cproduct
@cproduct
Jun 10 2015 19:38
testen was passiert, wenn das zu speichernde bild schon existiert (also das dither bild)
Patrick D.
@pichuio
Jun 10 2015 19:38
bei mir wirds überschrieben
cproduct
@cproduct
Jun 10 2015 19:39
ja ist das so gewünscht? wenn ja, muss mans auch testen
Patrick D.
@pichuio
Jun 10 2015 19:40
ahh ok...das hääte ich jetzt anders interpretiert
XD
cproduct
@cproduct
Jun 10 2015 19:41
was? :D
Patrick D.
@pichuio
Jun 10 2015 19:41
dass das getestet werden muss XD
cproduct
@cproduct
Jun 10 2015 19:42
normal schon. wenn man sich z.b. denkst, dass einer an der funktion rum schraubt und es dann nicht mehr überschreibt und abschmiert
Patrick D.
@pichuio
Jun 10 2015 19:43
hmm
Patrick D.
@pichuio
Jun 10 2015 20:16
lohnt sich der aufwand den mainprocess noch zu testen?
cproduct
@cproduct
Jun 10 2015 20:17
ich glaub nicht
Patrick D.
@pichuio
Jun 10 2015 20:21
wir haben jetzt 80% coverage
also line coverage
und 83% branch coverage...was auch immer das ist
Patrick D.
@pichuio
Jun 10 2015 20:29
jetzt verlieren die idioten auch noch gegen die usa