improve performance of scrollview in main screen #39

Closed
marc.nause wants to merge 5 commits from marc.nause/TinyWeatherForecastGermany:fixjerkylist into master

I have done some performance optimizations in order to improve the rendering speed of the ListView in the main screen.

Bitmaps are chached in memory. This speeds up access to the images but uses RAM instead. Since the number of images is relatively small, this tradeoff works well for me.

I have done some performance optimizations in order to improve the rendering speed of the ListView in the main screen. Bitmaps are chached in memory. This speeds up access to the images but uses RAM instead. Since the number of images is relatively small, this tradeoff works well for me.
marc.nause changed title from improve performance of scrollview in main screen to WIP: improve performance of scrollview in main screen 3 years ago
Poster

Cached bitmaps will be removed from cache if memory gets low.

Cached bitmaps will be removed from cache if memory gets low.
marc.nause changed title from WIP: improve performance of scrollview in main screen to improve performance of scrollview in main screen 3 years ago
Owner

Good idea! Let me take a closer look tomorrow, then I will merge. Thanks a lot for helping!

Good idea! Let me take a closer look tomorrow, then I will merge. Thanks a lot for helping!
Owner

Thanks a lot, the performance gain is really remarkable! However, meanwhile I improved the ForecastBitmap class to display even more information in this bar, so there were conflicts.

I will close and not merge with codeberg, but I applied the whole pull request with a patch-file, this way it was easier for me to keep my changes and to resolve the conflicts :-)

Again, thanks a lot!!!

Thanks a lot, the performance gain is really remarkable! However, meanwhile I improved the ForecastBitmap class to display even more information in this bar, so there were conflicts. I will close and *not* merge with codeberg, but **I applied the whole pull request with a patch-file**, this way it was easier for me to keep my changes and to resolve the conflicts :-) Again, thanks a lot!!!
Starfish closed this pull request 3 years ago
Please reopen this pull request to perform a merge.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b marc.nause-fixjerkylist master
git pull fixjerkylist

Step 2:

Merge the changes and update on Forgejo.
git checkout master
git merge --no-ff marc.nause-fixjerkylist
git push origin master
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Starfish/TinyWeatherForecastGermany#39
Loading…
There is no content yet.