Skip to content
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

We're losing performance around 500 lines, and around 800 lines is almost impossible #82

Open
rbatty19 opened this issue Sep 6, 2024 · 2 comments

Comments

@rbatty19
Copy link

rbatty19 commented Sep 6, 2024

I was trying to do the year's expense accounts, I'm using about 400 lines

I was wondering how many calculations I can do on the same page and I found a drop in performance at 500 lines, and when I get to 800 lines the drop in performance is drastic

@bornova
Copy link
Owner

bornova commented Sep 6, 2024

Hi @rbatty19,
Yes, this is a limitation unfortunately and I have been tinkering about a solution for a while now. It's not only the number of lines but the expression being calculated that is the bottleneck. Even a single line expression can crash the app (Ex. 1:9999999). The app calculates expressions line by line and displays the results of the calculations after each line is calculated. Since a certain amount of time is needed to calculate each line, more lines mean decreased performance. And if the expressions are especially complex, the performance hit compounds quickly.

I am working on a interim solution where the calculations can be done in the background so that at least the UI does not crash but it may be a while till it's implemented.

Thank you for the feedback.

@bornova
Copy link
Owner

bornova commented Sep 16, 2024

Hi @rbatty19,
I just released v5.3.2 with some performance improvements. Can you please try the new version and let me know if you see any improvement?

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

No branches or pull requests

2 participants