clementine-player / Clementine

:tangerine: Clementine Music Player
https://www.clementine-player.org/
GNU General Public License v3.0
3.72k stars 671 forks source link

Songs in cuesheet plays beginning of the next song #6698

Open duncanyuen opened 4 years ago

duncanyuen commented 4 years ago

Before posting

Please follow the steps below and check the boxes with [x] once you did the step.

System information

Please provide information about your system and the version of Clementine used.

Expected behaviour / actual behaviour

However, the problem does not persist if it next song in the playlist is the next track from the source file.

Steps to reproduce the problem (only for bugs)

  1. Import tracks from a cuesheet
  2. Arrange the tracks such that the next song in queue is not the next track from the cuesheet source file

Notes

It seems that this issue (https://github.com/clementine-player/Clementine/issues/3492) is referencing the same problem, but the solution points to a dead link.

FredolDol commented 3 years ago

Hi, nobody will try to solve this?

thehta commented 2 years ago

Hey, still experiencing this same issue. Here is the debug output from version 1.4.0rc1-827-gbbda59a5f when this issue occurs.

# Start playing song from Clementine playlist

10:25:08.954 INFO  MoodbarLoader:110                Loading cached moodbar data for "/music/Death Grips/The Money Store (2012)/The Money Store.flac"
10:25:08.956 DEBUG PlaylistBackend:293              Saving playlist 2
10:25:08.971 DEBUG GstEnginePipeline:877            Decoder bin pad added: src_0
10:25:08.971 DEBUG GstEnginePipeline:890            Initial decoder caps: audio/x-raw, format=(string)S24_32LE, layout=(string)interleaved, rate=(int)96000, channels=(int)2, channel-mask=(bitmask)0x0000000000000003
10:25:08.971 DEBUG GstEnginePipeline:1308           Setting format to "F32LE"
10:25:08.989 DEBUG MainWindow:1596                  position: 0 , scrobble point: 85 , lastfm status: 0 , play count point: 85 , is local library item: true , playlist have incremented playcount:  false

# Scrobbling output removed for brevity

10:27:58.880 DEBUG MainWindow:1596                  position: 170 , scrobble point: 85 , lastfm status: 0 , play count point: 85 , is local library item: true , playlist have incremented playcount:  true

# Original song audio ends (at the time stipulated in CUE file). Next song in CUE playlist starts bleeding in.

10:28:01.053 DEBUG GstEnginePipeline:877            Decoder bin pad added: src_0
10:28:01.053 DEBUG GstEnginePipeline:890            Initial decoder caps: audio/x-raw, format=(string)S24_32LE, layout=(string)interleaved, rate=(int)96000, channels=(int)2, channel-mask=(bitmask)0x0000000000000003
10:28:01.053 DEBUG GstEnginePipeline:896            Ignoring native format since pipeline is already running.

# Next song in CUE (bleeding song) stops playing here, ~5 seconds after original song ended. Next song in Clementine playlist starts playing.

10:28:06.338 DEBUG GstEnginePipeline:657            New segment started, EOS will signal on next buffer discontinuity
10:28:06.339 DEBUG GstEnginePipeline:1039           Buffer discontinuity - emitting EOS
10:28:06.343 INFO  MoodbarLoader:110                Loading cached moodbar data for "/music/Death Grips/The Money Store (2012)/The Money Store.flac"
10:28:06.343 DEBUG PlaylistBackend:293              Saving playlist 2
10:28:06.362 DEBUG MainWindow:1596                  position: 0 , scrobble point: 93 , lastfm status: 0 , play count point: 93 , is local library item: true , playlist have incremented playcount:  false

Relevant CUE file:

FILE "The Money Store.flac" WAVE
  TRACK 01 AUDIO
    TITLE "Get Got"
    PERFORMER "Death Grips"
    INDEX 01 00:00:00
  TRACK 02 AUDIO
    TITLE "The Fever (Aye Aye)"
    PERFORMER "Death Grips"
    INDEX 01 02:51:60
thehta commented 2 years ago

It seems that this issue (#3492) is referencing the same problem, but the solution points to a dead link.

@duncanyuen You can find the source code here. Merging it in now is a different story though...