Skip to content

Comments

Cope with large logs#743

Merged
corrados merged 1 commit intojamulussoftware:masterfrom
pljones:feature/update-historytool
Nov 25, 2020
Merged

Cope with large logs#743
corrados merged 1 commit intojamulussoftware:masterfrom
pljones:feature/update-historytool

Conversation

@pljones
Copy link
Collaborator

@pljones pljones commented Nov 25, 2020

Minor update to historytool to handle logs with over 20,000 lines.

Mine just hit 20,044 and the graph stopped updating... There's a new 'last=true' parameter (default is the 'reverse' parameter value, which is passed as false when getting the data for the graph) to make sure it's the last 20,000 records from the log being used.

You can always use 'max_lines' instead but this now allows more careful tuning. (I probably don't need 20,000 records for my 300 day graph, for example.)

@corrados corrados merged commit b263109 into jamulussoftware:master Nov 25, 2020
@pljones pljones deleted the feature/update-historytool branch November 25, 2020 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants