-
Notifications
You must be signed in to change notification settings - Fork 490
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
Balance assertion fails with lot-priced commodities #2318
Comments
I bisected it to this commit: ce8fec9 This broke my ledgers with extensive use of foreign currencies, with dated valuations on liabilities in order to correctly report gain and loss due to exchange rate fluctuation when expenses are incurred vs when payments are made, so I'm currently stuck on ledger-3.2.1. Perhaps it's fine to allow non-valued balance assertions (as in case A above, which I expected to fail), but it's a problem to reject the ledger when the units all match exactly (as in case B above, which I expected to work and which no longer works). |
This avoids the following regression: ledger/ledger#2318 Eventually I would like to see this either fixed upstream or worked around, but keeping an older version is easier for now.
ledger 3.3.2-20230330
Works, even though the units don't exactly match:
Doesn't work but should work:
ledger 3.2.1-20200518
Failed as expected because the units don't match:
Worked as expected because the units do match:
The text was updated successfully, but these errors were encountered: