Repeated import of cover art from files

Bug #1735996 reported by Uwe Klotz
4
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Triaged
Medium
Unassigned

Bug Description

Cover art for the same track might be requested from UI components like widgets in the library, the cover art display on the deck and the spinnies almost concurrently. The cover art cache accepts multiple requests for the same track from different requestores if the cover is not cached already, although all requests reference the same file. Why? Because the requestId contains both the requestor and the cover hash.

As a consequence multiple subsequent read operations for the same cover art image are performed. For embedded cover art this becomes obvious from the logs, because a SoundSourceProxy is created and embedded cover art is parsed repeatedly from the audio file, e.g. when loading a track into a deck.

Pending requests for the same cover hash need to be combined instead of creating new requests.

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

Should IMHO be fixed before releasing 2.1.0.

Changed in mixxx:
milestone: none → 2.1.0
description: updated
description: updated
summary: - Repeated reloading of cover art
+ Repeated import of cover art from files
Changed in mixxx:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Daniel Schürmann (daschuer) wrote :

The rejected PR for the cover hash might be related.
https://github.com/mixxxdj/mixxx/pull/996

Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

Both issues are unrelated, although targeting overlapping parts of the code.

Changed in mixxx:
assignee: nobody → Uwe Klotz (uklotzde)
status: Confirmed → In Progress
Revision history for this message
Uwe Klotz (uklotzde-deactivatedaccount) wrote :

I won't be able to complete this in time. It turned out that the caching and hashing needs to be changed to implement this properly. Maybe has more dependencies with https://github.com/mixxxdj/mixxx/pull/996 than expected.

Changed in mixxx:
assignee: Uwe Klotz (uklotzde) → nobody
milestone: 2.1.0 → 2.2.0
importance: Medium → Low
importance: Low → Medium
Revision history for this message
Be (be.ing) wrote :

Uwe or Daniel, would one of you like to assign yourself to take this up for 2.2?

Be (be.ing)
Changed in mixxx:
milestone: 2.2.0 → none
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Due to lack of progress, marking Triaged and clearing assignee. Feel
free to revert if it is in fact still in progress :).

Changed in mixxx:
status: In Progress → Triaged
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/8991

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.