-
Notifications
You must be signed in to change notification settings - Fork 70
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
Count of stocks is sometimes a float value, could we round it? #5
Comments
I don't think this is a mistake. TR allows you to buy fractional shares, that is why the count is a float. You probably bought a fractional share |
No, I never buy fractional shares. Really. Either it is TR which somehow wants to be smart and include the fee, or something else. But also here I might do a breakpoint next time to check out what the unprocessed incoming data is. For above: I know for 1000% sure I bought 20 Agnico, 1 Amazon, 5 Deutsche Boerse (no fraction), 50 TF1 and 100 Schaeffler (no fraction). |
If you buy through the Savings feature which allows to buy every month for example for 100e of one action then it will buy the necessary amount of shares to match the monthly amount defined, likely not an integer. Did you use that maybe? |
NO, really not. Sorry for the late reply. I only buy full (integer) shares. The only time I got fractional shares was once when Stellantis gave out "free papers", but I sold them. |
So to settle this, too: This was once the result: Now it is: The "cashChangeAmount": 1114.48, is correct.
|
I think the time line event is not sufficiently detailed. The number if shares is missing so the tax can not be calculated. or the other way: shares can not be calculated without knowing the fees/taxes. Is there a way to access fees/taxes/shares with the API? |
When trying to import myTransactions.csv by "Datei - Importieren" to Portfolio Performance, via:
some stocks in the CSV file have a float amount, e.g. 19,7505 Morphosys. I guess it should normally be 20.
I don't know exactly why, and I guess the fault is for TR, it could be they somehow include their fee in this calculation or so.
A workaround could be to round this number (really, otherwise 19,75 would become 19, which is wrong, but e.g. 19,3 should be 19). This could happen by default or if you really wish you could turn it off by parameter.
Extract to reproduce this:
Additionally we should enforce to use either . or , when creating the CSV. After importing I was triple-millionaire :)
The text was updated successfully, but these errors were encountered: