New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enhancement dividend as event referring to #697 #699

Closed
wants to merge 3 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@cmaoling
Contributor

cmaoling commented Jan 9, 2017

No description provided.

cmaoling added some commits Jan 9, 2017

@buchen

This comment has been minimized.

Owner

buchen commented Jan 10, 2017

Wenn es Dir darum geht, die Dividendenzahlungen ebenfalls mit ins Chart aufzunehmen, dann wäre es mir lieber die in der Klasse "SecuritiesChart" auszulesen und direkt "addMarkerLines" aufzurufen. Warum vorher einen SecurityEvent anlegen?

In einem zweien Schritt sollten wir überlegen, ob man das Ein/Ausblendbar machen kann. Wenn ich Diagramme wie dieses hier sehe, dann ist das schon nicht mehr hilfreich.

In die Klasse SecurityEvent gehört das so (zumindest die konkrete Zahl) nicht rein. Erstens wird das mit in dem XML persistiert. Damit hätte man aber die Dividendenbuchung mehrfach - was wenn man die Buchung editiert. Zweitens sollten die SecurityEvents unabhängig von einer Buchung. Also das technische Ereignis "Dividende" kann ich mir so vorstellen. Dann aber auch, wenn ich das Wertpapier gar nicht besitze und vielleicht die Performance inklusive der potentiellen Dividendenzahlungen vergleichen möchte. Auf Basis dieses "Dividenden Events" könnte man dann vorschlagen eine Dividendenbuchung zu erfassen. Oder ähnliches. Das ist aber eine grössere Geschichte. Von Yahoo Finance könnte man diese Dividenden-Events bekommen.

@cmaoling

This comment has been minimized.

Contributor

cmaoling commented Jan 10, 2017

Abend.
Vielleicht habe ich übersehen das die Information bereits in der Security Chart verfügbar ist. Als Securityevent, weil sie damit sogleich im Eventtab aufgenommen wurden, daß macht für mich Sinn.

Das Ein- und Ausschalten kann man sich überlegen. Hatte auch schon gedacht, das konfigurierbar zu machen. Wollte aber erstmal diese Änderung einbringen....

Ok, erwischt. Die Persistent im XML hat mich sogar sehr erfreut. Dein dritter Absatz beschreibt ziemlich genau was ich am Ende einbauen will. 🥇
Wollte die Event zunächst von ariva holen und nicht yahoo, aber mehr aus Gewohnheit.
Aber die historischen Event s sollen m.E. der Security als Security Event zugeordnet sein und unabh. Von einer konkreten Buchung. Dann entsprechend eine Funktion, die Events für eine Tag extrahiert (das würde entsprechend bei den Solist auch eine Editierbarkeit ermöglichen) und daraus dann eine Buchung generiert.... Ein bisschen wie bei den Sparplänen... Aber soweit ja e Ich das noch nicht durcharchitekturiert.
Ich persönlich würde diese Buchungen von Umsätzen mir dann komplett automatisch generieren lassen und anhand der realen Buchungsbelege bei Bedarf editieren (eigentlich immer nur Steuerabzug..). Das PDF importieren wäre eine Alternative.
Da ich aber aktuell einen Weg suche in PP Aktienhandel aus den letzten 15 Jahren zu importieren, habe ich eigentlich keine oder nur ein Sunset an PDFs von einer der unterstützen Banken....

@buchen

This comment has been minimized.

Owner

buchen commented Jan 11, 2017

Vielleicht habe ich übersehen das die Information bereits in der Security Chart verfügbar ist.

Im Chart sind die natürlich noch nicht, aber genauso wie ich aus den (Ver)Kaufsbuchungen marker lines generiere, könnte man ja aus den Dividendenbuchungen ebenfalls marker lines generieren - eben aus den AccountTransactions. In der Klasse Security gibt es auch eine Methode getTransactions mit der man alle zu diesem Wertpapier gehörenden Buchungen bekommt.

Ok, erwischt. Die Persistent im XML hat mich sogar sehr erfreut. Dein dritter Absatz beschreibt ziemlich genau was ich am Ende einbauen will. 🥇

Deine Idee ist auch gut. 😄 Ariva könnte vielleicht schwierig werden wegen der Nutzungsbedingungen - die habe ich jetzt allerdings nicht im Kopf. Aber bei Yahoo Finance gibt es die Informationen + PP ruft ja schon Kurse ab. Man müsste mit einem weitere Flag die Dividendeninformationen abfragen und verarbeiten. Der Kurse-Update hat über die Jahre etwas technical debt angesammelt, darum ist das vielleicht nicht ganz gerade aus.

Wenn man das hat, kann man in weiteren Schritten:

  • Buchungen vorschlagen (erkennen ob ein Event neu ist), ggf. auch aus Buchungen einen Event generieren falls Yahoo das nicht hat?
  • Die Benchmark Datenreihe optional um Buchungen erweitern
  • Oder alternativ einen "adjusted close" analog zu Yahoo berechnen

...

In diesem Change würde ich jetzt erst mal die Dividenden Marker Lines unabhängig von SecurityEvent einbauen.

@cmaoling

This comment has been minimized.

Contributor

cmaoling commented Jan 11, 2017

Abend. Um hier doppelte Arbeit zu vermeiden eine sprachliche Klarstellung:
"In diesem Change würde ich jetzt erst mal ..". Das klingt für mich, als würdest Du meinen PR nehmen wollen und die Änderung bzgl. Unabhängigkeit vom SecurityEvent selbst umsetzen. Oder ist das die Bitte/Aufforderung an mich den PR nachzubessern? Halte Deine Idee für valide und kann das machen...

@buchen

This comment has been minimized.

Owner

buchen commented Jan 14, 2017

Abend. Um hier doppelte Arbeit zu vermeiden eine sprachliche Klarstellung:
"In diesem Change würde ich jetzt erst mal ..".

Guter Punkt - da war ich unsauber. Ich mache jetzt erst mal nichts (bin sowieso mit PDF Dokumenten und anderen GUI Bugs beschäftigt)

@cmaoling

This comment has been minimized.

Contributor

cmaoling commented Jan 16, 2017

@buchen: "In diesem Change würde ich jetzt erst mal die Dividenden Marker Lines unabhängig von SecurityEvent einbauen." - Done.

buchen added a commit that referenced this pull request Jan 22, 2017

@buchen

This comment has been minimized.

Owner

buchen commented Jan 22, 2017

Hi @cmaoling, ich habe mir per Cherry pick den letzten Change geholt und in den master gemergt. Die Änderungen am SecurityEvent habe ich erst mal nicht mit aufgenommen - wie oben schon gesagt, habe ich das Problem mit der aktuellen Implementierung, dass es die Dividendenbuchung doppelt persistiert und nicht editierter ist, etc. Abgesehen fehlt ja noch mehr drumherum das man mit den erweiterten SecurityEvent was angefangen kann --> separater Branch.

Auf die Change habe ich jetzt ein kleines Konfigurationsmenü gesetzt. Dann kann man wählen ob man die Dividenden sehen möchte oder nicht. Gerade bei meinen amerikanischen Wertpapiere, die einmal im Quartal ausschütten, können das ganz schön viele Linien sein.

@buchen buchen closed this Jan 22, 2017

cmaoling added a commit to cmaoling/portfolio that referenced this pull request Jan 25, 2017

@cmaoling cmaoling referenced this pull request Mar 20, 2017

Closed

Enhancement event loader #731

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment