Bug 327691 - Investment performance report calculation
Summary: Investment performance report calculation
Status: RESOLVED FIXED
Alias: None
Product: kmymoney
Classification: Applications
Component: general (show other bugs)
Version: 4.6.4
Platform: unspecified Linux
: NOR wishlist
Target Milestone: ---
Assignee: KMyMoney Devel Mailing List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-11-16 17:13 UTC by Mattia
Modified: 2017-05-14 04:42 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 5.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Mattia 2013-11-16 17:13:35 UTC
Investment performance report should be calculated using the purchase date for every single investment (start: purchase date of investment - end: today).

Reproducible: Always

Actual Results:  
Actual investment performance use a user defined data range common for all investments and this leads to wrong annualized return calculation.
Comment 1 allan 2013-11-16 20:22:26 UTC
But what if you want to compare stocks purchased on different dates?

What if you want to see how stocks performed over say the last year or the last three years?

Your way, you might get an accurate figure which does not give you what you need.
Comment 2 Mattia 2013-11-17 09:00:34 UTC
(In reply to comment #1)
> But what if you want to compare stocks purchased on different dates?
> 
> What if you want to see how stocks performed over say the last year or the
> last three years?
> 
> Your way, you might get an accurate figure which does not give you what you
> need.

I see your point. Well, the solution would be to create a new report, say "investment performance by investment", that doesn't have a start date common for every investment, but uses the first purchasing date of every single investment. This is actually not possible to do in a single report for all investments.

The existing "investment performance by account" and "investment performance by type" should also being renamed using the suffix "this year".
Comment 3 allan 2013-11-17 10:39:00 UTC
On 17/11/13 09:00, Mattia wrote:
> https://bugs.kde.org/show_bug.cgi?id=327691
>
> --- Comment #2 from Mattia <mattia.verga@tiscali.it> ---
> (In reply to comment #1)
>> But what if you want to compare stocks purchased on different dates?
>>
>> What if you want to see how stocks performed over say the last year or the
>> last three years?
>>
>> Your way, you might get an accurate figure which does not give you what you
>> need.
>
> I see your point. Well, the solution would be to create a new report, say
> "investment performance by investment", that doesn't have a start date common
> for every investment, but uses the first purchasing date of every single
> investment. This is actually not possible to do in a single report for all
> investments.
>
> The existing "investment performance by account" and "investment performance by
> type" should also being renamed using the suffix "this year".
>

The best thing, then, is to change the bug to a wishlist item.  At the 
foot of the bug report, change the severity to Wishlist.
Comment 4 wojnilowicz 2017-05-14 04:42:08 UTC
Git commit b3fd5b0f41ab82b459e7382710ddac9aa2796ae8 by Łukasz Wojniłowicz.
Committed on 14/05/2017 at 04:38.
Pushed by wojnilowicz into branch 'master'.

Add option for choosing input for investment calculations

Performance and Capital Gains reports get option for choosing: Sold,
Owned or both investment sums to do calculations. This allows to measure
precisely and separately performance of current and sold investments.
FIXED-IN:5.0

Differential Revision: https://phabricator.kde.org/D5678

M  +32   -0    kmymoney/dialogs/kreportconfigurationfilterdlg.cpp
M  +1    -0    kmymoney/dialogs/kreportconfigurationfilterdlg.h
M  +24   -6    kmymoney/mymoney/mymoneyreport.cpp
M  +11   -0    kmymoney/mymoney/mymoneyreport.h
M  +13   -10   kmymoney/reports/listtable.cpp
M  +368  -270  kmymoney/reports/querytable.cpp
M  +4    -1    kmymoney/reports/querytable.h
M  +1    -1    kmymoney/widgets/CMakeLists.txt
M  +50   -26   kmymoney/widgets/reporttabcapitalgain.ui
M  +28   -0    kmymoney/widgets/reporttabimpl.cpp
M  +12   -0    kmymoney/widgets/reporttabimpl.h
C  +11   -38   kmymoney/widgets/reporttabperformance.ui [from: kmymoney/widgets/reporttabcapitalgain.ui - 057% similarity]

https://commits.kde.org/kmymoney/b3fd5b0f41ab82b459e7382710ddac9aa2796ae8