Skip to content

Calcolo del TWRR#2

Open
lordeko wants to merge 1 commit into
ilbonte:mainfrom
lordeko:lordeko-patch-1
Open

Calcolo del TWRR#2
lordeko wants to merge 1 commit into
ilbonte:mainfrom
lordeko:lordeko-patch-1

Conversation

@lordeko

@lordeko lordeko commented Feb 3, 2025

Copy link
Copy Markdown

Implementa il calcolo del TWRR. Se il range è inferiore a 365 giorni non annualizza il calcolo, poi lo fa su base 365

Implementa il calcolo del TWRR. Se il range è inferiore a 365 giorni non annualizza il calcolo, poi lo fa su base 365
@ilbonte

ilbonte commented Feb 3, 2025

Copy link
Copy Markdown
Owner

Grazie 1000 per le PR! Le guardero' domani, oggi sono un po' occupato :)

@ilbonte

ilbonte commented Feb 4, 2025

Copy link
Copy Markdown
Owner

@lordeko Ho guardato la formula ma non mi torna perche' al denominatore fai
day.cumulativeInvestment + portfolioData[0].patrimonio.

Forse non ti sei accorto che cumulativeInvestment e' il comulativo dei versamenti fatti a Directa fino al determinato giorno, ma non e' detto che siano stati tutti investiti.
Il TWRR dovrebbe considerare solo quanto investito, ti torna?

Inoltre non ho capito perche' sommi il patrimonio del primo giorno.
FYI: il patrimonio comprende sia la liquidita' che quanto investito.

@lordeko

lordeko commented Feb 4, 2025

Copy link
Copy Markdown
Author

Ciao @ilbonte, io sarei dell' opinione di inserire la liquidità nel TWRR, proprio perché il patrimonio è liquidità+investimenti. Io lo vedo come un asset a rendimento 0% (se euro ovviamente).
Inoltre è proprio nella logica stessa del twrr considerarla, non è detto che io investa i soldi nel giorno in cui li metto nel "fondo" oppure potrei decidere di stare liquido in un momento di stress dei mercati.
Per il fatto di aver aggiunto il patrimonio del primo giorno era perché questo è riscalato a 0 e mi pare facesse casino nei numeri se non lo mettevo, dovrei rivedere

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants