Skip to content

chore(benchmark): add warning for probably low memory #5797

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

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

kisvegabor
Copy link
Member

fixes #5654

Description of the feature or fix

A clear and concise description of what the bug or new feature is.

Notes

@FASTSHIFT
Copy link
Collaborator

FASTSHIFT commented Mar 6, 2024

We may need to consider reducing some memory usage, because the 128K memory pool may not be available on most microcontrollers.
I checked and it was caused by the container_with_scrolling_cb function calling card_create 50 times to create 50 widgets.
Can we reduce the number of widgets, like 25? Or do we choose the number of widgets based on the size of the screen?

@kisvegabor
Copy link
Member Author

kisvegabor commented Mar 11, 2024

I checked and it was caused by the container_with_scrolling_cb function calling card_create 50 times to create 50 widgets.
Can we reduce the number of widgets, like 25? Or do we choose the number of widgets based on the size of the screen?

It's fixed 50 to test the performance with many widgets. If we reduced it to 25 we were beat the purpose of the test.

@FASTSHIFT FASTSHIFT merged commit 2722e39 into lvgl:master Mar 12, 2024
@kisvegabor kisvegabor deleted the chore/benchmark branch March 12, 2024 13:44
HongChao6 pushed a commit to HongChao6/lvgl that referenced this pull request Oct 18, 2024
HongChao6 pushed a commit to HongChao6/lvgl that referenced this pull request Oct 18, 2024
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.

Issue with Benchmark Execution in Version 9.0.0 - LV_MEM_SIZE Adjustment Required
4 participants