imp: reg: areg: Align to first 1000 postings by default, rather than 100.

This has the same performance characteristics for a big decrease in alignment
issues. 1000 postings seems to still put it under the memory bottleneck.
This commit is contained in:
Stephen Morgan 2022-03-12 09:57:57 +11:00 committed by Simon Michael
parent 129f37cfe6
commit cbdd86f4dc
3 changed files with 3 additions and 3 deletions

View File

@ -48,7 +48,7 @@ Each `aregister` line item shows:
Transactions making a net change of zero are not shown by default; Transactions making a net change of zero are not shown by default;
add the `-E/--empty` flag to show them. add the `-E/--empty` flag to show them.
For performance reasons, column widths are chosen based on the first 100 lines; For performance reasons, column widths are chosen based on the first 1000 lines;
this means unusually wide values in later lines can cause visual discontinuities this means unusually wide values in later lines can cause visual discontinuities
as column widths are adjusted. If you want to ensure perfect alignment, as column widths are adjusted. If you want to ensure perfect alignment,
at the cost of more time and memory, use the `--align-all` flag. at the cost of more time and memory, use the `--align-all` flag.

View File

@ -25,7 +25,7 @@ $ hledger register checking
With --date2, it shows and sorts by secondary date instead. With --date2, it shows and sorts by secondary date instead.
For performance reasons, column widths are chosen based on the first 100 lines; For performance reasons, column widths are chosen based on the first 1000 lines;
this means unusually wide values in later lines can cause visual discontinuities this means unusually wide values in later lines can cause visual discontinuities
as column widths are adjusted. If you want to ensure perfect alignment, as column widths are adjusted. If you want to ensure perfect alignment,
at the cost of more time and memory, use the `--align-all` flag. at the cost of more time and memory, use the `--align-all` flag.

View File

@ -268,7 +268,7 @@ postingsOrTransactionsReportAsText alignAll opts itemAsText itemamt itembal repo
mconcat . snd $ mapAccumL renderItem (startWidth amt, startWidth bal) itemsWithAmounts mconcat . snd $ mapAccumL renderItem (startWidth amt, startWidth bal) itemsWithAmounts
where where
minWidth = 12 minWidth = 12
chunkSize = 100 chunkSize = 1000
renderItem (amtWidth, balWidth) item@(_, amt, bal) = ((amtWidth', balWidth'), itemBuilder) renderItem (amtWidth, balWidth) item@(_, amt, bal) = ((amtWidth', balWidth'), itemBuilder)
where where