plattenalbum icon indicating copy to clipboard operation
plattenalbum copied to clipboard

Album loading performance

Open SoongNoonien opened this issue 2 years ago • 3 comments

The album(-cover) loading performance is improvable. This issue is to provide a place for discussion on that topic. It has been extracted from #39.

SoongNoonien avatar Aug 14 '21 21:08 SoongNoonien

The newest commit (4245b40b3d81a44d97ba92a62c96f5ab5993a633) changes the album loading a bit. Initial GUI blocks in combination with large artists (many albums) should now be gone. I also separated the fetching/locating of the covers from the actual loading of the albums which decreases the initial time needed to display the albums without their covers. Sadly the overall time did not really decrease. The complete procedure is now run in a separate thread using python's threading module but this should manly increase the reliability and not the performance.

@dgcampea It would be really nice if you could test these changes.

SoongNoonien avatar Sep 09 '21 14:09 SoongNoonien

I can confirm that 4245b40 in general improves the responsiveness of the GUI (previously it would stutter when changing between album artists).

The listing of "all artists" still takes a while but the times seem to have gone down as well.

dgcampea avatar Sep 11 '21 12:09 dgcampea

Might be worth a look (haven't looked too deeply into it yet):

  • https://github.com/begnac/ampd
    • https://github.com/begnac/gampc

dgcampea avatar Oct 04 '21 09:10 dgcampea