Posts: 1,515
Joined: Mar 2018
Reputation:
34
Great, thanks for bringing this up.
Posts: 1,515
Joined: Mar 2018
Reputation:
34
2023-06-08, 16:20
(This post was last modified: 2023-06-08, 20:09 by Buschel. Edited 1 time in total.)
I did a change and hope to get a TF build out soon. I found one bug in the code and also saw reduced resolution when rolling this back to what was used in the last TF build. I hope this is fixed then for you as well. Why loading takes ages I cannot tell. But I remember you reported something like this before, so it might be related to your setup. Most of the thumbs will now be reloaded.
Posts: 587
Joined: Apr 2014
Reputation:
10
Sounds good. From what I could tell when I let it sit earlier was that it simply stopped loading the high resolution art after the first couple of rows and never resumed. So I was stuck with low resolution for the rest of them no matter how long I let it sit there and wait.
For the other issue with art loading that was just slow loading at all but not anywhere near what I’m experiencing now.
Posts: 1,515
Joined: Mar 2018
Reputation:
34
1.13 build 3993.44
Next test build to resolve stuttering when scrolling, this now should provide high resolution thumbs again (if available). You might need to remove the image cache. This can be done via removing the app complete and reinstall, or via enabling the setting „remove cache at start“ and then restarting the app.
Posts: 587
Joined: Apr 2014
Reputation:
10
Thanks Buschel,
Testing will have to wait until Monday for me as I will be out of town. Will report back then!
Posts: 1,515
Joined: Mar 2018
Reputation:
34
2023-06-09, 12:01
(This post was last modified: 2023-06-10, 14:01 by Buschel. Edited 1 time in total.)
Needed to dig further into this and found one more bug which is still present in 1.13 build 3993.44. For testing you will need to remove the cache, then test (all images should have the desired high resolution and scrolling should not stutter), but this will not persist. After restart the images are still of high quality, but stuttering will be back. Reason is, that the downscaled images are not persisted properly.
I already updated the PR, but it will not be able to push another TF before next week.
Edit: The sdwebimage migration needed a lot more tweaking than I thought, but I am confident this is in good shape now. The cache paths changed and the file naming changed. The rework also let me recognize some outdated clear-cache implementation in the App which I now updated and recognize an improvement opportunity for playlist thumbs (which should have a higher quality as also pre-scaled cached images are used).
Posts: 587
Joined: Apr 2014
Reputation:
10
Thanks for putting in the unexpected extra work on this.
So to be clear, am I needing to waiting for the next build before testing?
Posts: 1,515
Joined: Mar 2018
Reputation:
34
1.13 build 3993.45
This has all the latest rework for image scaling and cacheing. If you had installed and used one of the former image cache related builds, please clean the cache (can be done in settings and then restart the app). If you do not delete the cache, you might still see stuttering as the downscaled images were not cached properly and only full-size images were added to cache.
Posts: 587
Joined: Apr 2014
Reputation:
10
2023-06-16, 04:41
(This post was last modified: 2023-06-16, 04:45 by amasephy. Edited 1 time in total.)
Just tested on iPhone. I’m still getting a little bit of stutter on tv show season - episode lists. Even in an instance where there is no season poster art.
I can post video showing it if you need it. Will report back on iPad tomorrow.
Edit: Stutter only happens in that one view when rounded corners are enabled. Disabled it’s gone entirely.