Skip to content
New issue

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

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

Already on GitHub? # to your account

Added new dashboard widget: inflation-adjusted internal rate of return #1120

Closed
wants to merge 1 commit into from

Conversation

FelixHertlein
Copy link

No description provided.

@buchen
Copy link
Member

buchen commented Feb 19, 2019

Hi @FelixHertlein,

super Idee! Die Inflationsrate unterschätzt man ja schnell. Das ist eine super Erweiterung.

Jetzt haben sich allerdings zwei Entwicklungen überschnitten. Mit #1110 hat @Ragas13 die harmonisierten Verbraucherpreise der europäischen Union (Eurostat) eingebaut - siehe Infos bei der EZB. Das ist super, weil es jetzt mehr als nur eine Datenreihe für Verbraucherpreise gibt. Typischen Nachfragen aus Österreich, der Schweiz, den Niederlanden war immer: wie kann ich andere Verbraucherpreise einpflegen? Das ganze ist als QuoteFeed implementiert, d.h. als ein (Pseudo)Wertpapier das man als Benchmark den entsprechenden Diagrammen hinzufügen kann. Siehe auch Alle Wertpapiere -> (+) Plus Button [oben rechts].

Was heißt das für Deinen Change? Ich denke man müsste den "Benchmark" frei wählen können. Dann kann man den bisherigen Verbraucherpreisindex (vom deutschen statistischen Bundesamt) nutzen, oder einen der harmonisierten Indices. Vermutlich würde man eine DataSeriesConfig an eine Subklasse von dem IndicatorWidget hängen das Du schon nutzt, und den Benutzer da den Benchmark auswählen lassen. Hast Du Zeit das noch zu ändern? Ansonsten könnte ich das on top von Deiner Change machen - es dauert aber vermutlich weil ich bald länger unterwegs bin.

Andreas.

@FelixHertlein FelixHertlein closed this by deleting the head repository Apr 3, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants