Comments (6)
Thanks for the report. There was one fix for roi
in hledger 1.33:
roi now correctly interacts with --value. #2190 (Dmitry Astapov)
Could you confirm your issue still happens with 1.33 ?
from hledger.
Thanks - I get the same results with hledger 1.33.1-gba1b40e50-20240502, windows-x86_64
from hledger.
@adept, any thoughts ?
from hledger.
@hbwales , in this transaction:
2020-12-01 Test1
A +200.00 ; date:2020-12-01
B +100.00 ; date:2021-01-01
C -300.00
your assumption seems to be that these cash movements have taken place:
- 200 from C to A
- 100 from C to B
However, roi takes an "A-centric" view, and treats all transactions that mention A as related to the investment. So the interpretation is this:
- 300 from C to A (money moving in, as C is not a PnL account)
- 100 from A to B (money moving out, as B is not a PnL account)
This is documented in https://hledger.org/1.33/hledger.html#semantics-of---inv-and---pnl
If this interpretation does not suit you, you need to split this transaction into two so that the transaction that is not related to your investment will not have any mention of A.
This is not a recent development, I would've expected roi
to behave like this from the very beginning
from hledger.
@hbwales, shall we close this ?
from hledger.
Closing, presumed resolved.
from hledger.
Related Issues (20)
- date2-format for date2, CSV import HOT 1
- Project level default configuration should be possible without user intervention HOT 6
- Inconsistent Decimal Mark Handling after CSV Import with Rules HOT 2
- 1.25: balance reports' HTML output stopped showing digit group marks HOT 1
- `stack8.10.yaml`, build failure `NonEmpty` does not export `singleton` HOT 3
- incorrect prices for "daily" roi HOT 8
- Install instruction under Linux with release 1.34 HOT 1
- tldr clients and documentation HOT 4
- Account type tag inconsistently recognized when declared in multiple files HOT 8
- `check` command should not stop after the first error HOT 4
- `--historical` uses period-end valuations of prices, while `--cumulative` doesn't HOT 15
- Ledger file formatting tool HOT 3
- Importing CSV: Creating complex `*.csv.rules` needs clarity HOT 6
- Code for Postings HOT 3
- ledger import not assigning default account2 as per manual HOT 3
- aregister -P, -C and -U miscalculate initial balance HOT 5
- UI bug in hledger web - autocomplete not working on the 5th account field and beyond HOT 1
- NBSP parsing seems to be broken (at least in CSV) HOT 1
- "every Nth day" periodic rule violating start date HOT 6
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from hledger.