search

portfolio-performance / portfolio

Track and evaluate the performance of your investment portfolio across stocks, cryptocurrencies, and other assets.
http://www.portfolio-performance.info
Eclipse Public License 1.0
2.94k stars 606 forks source link

Offene Enden beim PDF Import #197

Closed buchen closed 9 years ago

buchen commented 9 years ago

Im Nachgang zu Pull Request #185 müssen wir noch einige Änderungen machen, damit wir den Comdirect PDF Import überhaupt ausliefern können:

buchen commented 9 years ago

bildschirmfoto 2014-11-20 um 19 50 47

buchen commented 9 years ago

@simpsus vielleicht sollte man auch checken ob es sich überhaupt um ein Dokument der comdirect Bank handelt. Im unteren Bereich steht ja mindestens einmal comdirect bank - zumindest in der Beispieldatei. Dann ist die Fehlermeldung einfacher zu schreiben...

simpsus commented 9 years ago

neuer branch neues Glück. Oder auch: neuer Branch alte Probleme. Im Moment bekomme ich 12 Errors, die ich nicht verstehe:

image

Hast Du ne Idee?

simpsus commented 9 years ago

Zu den Tasks:

Fehleranzeige. Die Exceptions haben wir in der Errors Liste. Willst Du, dass ich hier versuche eine Anzeige zu machen, oder machst Du nen Rahmen den ich ausbauen kann?

ING-Diba Code. Das ist der dummy Extractor von Dir und der Menueintrag. Mehr hab cih noch nicht gesehen. Der branch von @tamueller ist bestimmt hierzu kompatibel... Wenn er nicht kommt bis wir hier fertig sind sollte man den Menueintrag vielleicht auch raus nehmen.

Strings: Sollte einfach sein

EUR: Momentan ist das der Weg. Der Charme daran ist, dass ich nach diesem Wort suchen kann denn das ist da immer als Trennzeichen. Keine Ahnung wie viele Wörter die Namen der Securities haben. Kann PP nicht eh nur €? Wenn das Feature gewünscht wird kann ich anhand einer Beispieldatei bestimmt versuchen das genereller zu machen. Ich hab im moment keine.

buchen commented 9 years ago

Hast Du ne Idee?

Refresh? Das ist eine neue Klasse (durch den Merge im Master Branch). Vielleicht hat Git die Dir untergeschobene und Eclipse sieht die noch nicht.

Willst Du, dass ich hier versuche eine Anzeige zu machen, oder machst Du nen Rahmen den ich ausbauen kann?

Ich dachte wir zeigen vielleicht eine weitere Tabelle mit "Extracting" Fehlern. Das kann ich einbauen. Oder Du probierst Dich daran?

Das ist der dummy Extractor von Dir und der Menueintrag.

Richtig. Den müsste man rausnehmen. Lass uns noch etwas warten. Einfach so mergen möchte ich den ING-Diba Code nicht.

Kann PP nicht eh nur €?

Na ja, ich mache ja keine Aussage über die Währung. Es kann also auch alles in Schweizer Franken sein. Im "feature_currencies" branch habe ich das mal so gebaut, dass man am Anfang entscheiden muss, auf welche Währung die Konten, Wertpapiere umgestellt werden. Dann kann man natürlich einzelnen Konten auch nachträglich ändern.

Aber: das ist hier ja (noch) nicht relevant, da eben Währungen nicht unterstützt werden.

EUR: Momentan ist das der Weg. Der Charme daran ist, dass ich nach diesem Wort suchen kann denn das ist da immer als Trennzeichen.

Okay. Muss das Konto in EUR sein? Oder auch die Dividende? Sprich: wenn man eine USD Dividende bekommt, wird dann ja trotzdem ein EUR Betrag gutgeschrieben. D.h. es funktioniert für alle in EUR geführten Konten?

Deine 50+ PDF Dokumente hast Du jetzt eingelesen und importiert, richtig? Musstest Du da viel nacharbeiten damit die Performance-Auswertungen wieder stimmen?

simpsus commented 9 years ago

Ich versuche mich mal an dem table. Hast Du eine Idee wo wir einen ähnlichen Table schonmal machen? Wenn ich aber nichtmal das Projekt zum kompilieren bekomme sag ich nochmal bescheid.

EUR: Ja, ich bekomme auch Gutschriften in USD und die werden mir als € gutgeschrieben. Ich parse mit Absicht nur die "Ergebniszeile" sodass das EUR sind und Deine Vermutung stimmt, das dass für alle in € geführten Konten funktioniert.

Meine Dokumente habe ich alle eingelesen (und damit ist die stabile Version von PP im Moment für mich keine Option mehr weil die Datei so viel weiter ist). Ich musste nicht eine Transaktion anpassen.

simpsus commented 9 years ago

So ich habe jetzt das mit der Anzeige "geschafft". Was gefällt mir da nicht:

Ich habe den branch gebranched aber leider habe ich schon wieder einen komischen merge gemacht (vorher waren die Files bei mir nicht zu sehen), deshalb habe ich mich nicht getraut einen Pull Request auf zu machen, wenn der wieder zu groß ist.

Mein Commit ist ae6dd1590a1e661558781f21d9f140e42bd0dd4e

tamueller commented 9 years ago

Hi.. sorry, dass ich quasi in der Versenkung verschwunden bin, habe gerade beruflich und privatlich zu viel um die Ohren. Mein Code funktioniert soweit, allerdings nicht für alle Nachrichten, momentan ist es glaube ich nur Kauf und Zinsen. Einzahlungen sind z.B. noch nicht drin. Wenn ihr den Code einmergen wollt, und fragen habt meldet euch. Ansonsten versuche ich einfach irgendwann ihn fertig zu bekommen, kann aber noch ein bisschen dauern.

buchen commented 9 years ago

@simpsus

Der Change passt doch. Ich habe ihn jetzt per cherry-pick in den feature_pdf_import branch reingezogen.

ich schon wieder einen komischen merge gemacht

Du hast anscheinend den branch von origin/master gemacht. Einfacher ist es, den branch auf origin/feature_pdf_import zu machen. Dann brauchst Du den merge gar nicht mehr.

Die Tabelle schließt unten nicht mit dem Dialog ab

Das ist ein blödes Verhalten von dem Eclipse Dialog. Da soll eigentlich der Progress Monitor auftauchen. Da der Job aber sowieso nicht im Hintergrund läuft (ich nutze jetzt einen "job"), bleibt der Bereich ungenutzt. Ich habe das Layouting jetzt etwas gesunde, so dass die Tabellen beim Resizing the Fensters entsprechend angepasst werden.

Die Tabelle wird immer angezeigt. Wenn es keine Probleme gibt sollte sie das nicht.

Das finde ich gar nicht so problematisch. Ich sage mal wir lassen das so.

buchen commented 9 years ago

@tamueller

Danke für die Rückmeldung. Ich schlage vor, wir lassen den INGDiba Extractor erst mal aussen vor. Ich selber kann so einen Import nicht durchspielen (ich habe keine INGDiba PDF Dokumente). Und dann kann ich mir vorstellen, dass von den ersten Verwendern direkt mal Fragen/Probleme kommen. Da braucht man auch Zeit. Wenn das soweit ist, dann kann ich recht schnell eine neue Version bauen und auf die Update Site stellen.