Skip to content

Conversation

comfyanonymous
Copy link
Owner

No description provided.

@comfyanonymous comfyanonymous merged commit 54a45b9 into master Jul 21, 2025
7 checks passed
@comfyanonymous comfyanonymous deleted the temp_pr branch July 21, 2025 18:19
sample_offset = int(sr * seek_time)
num_samples = int(sr * duration) if duration >= 0 else -1

# Small negative offset for MP3 artifacts, NOTE: this is LLM code so idk if it's actually necessary'
Copy link
Contributor

@bigcat88 bigcat88 Jul 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://huggingface.co/spaces/facebook/MusicGen/blob/23fe4834cd3c55beb42f9b9c68a5a34fc6644a25/audiocraft/data/audio.py#L89-L91

it will be more accurate to just skip one sample than to skip 0.1 second(but 0.1 will be enough for most of mp3, but will not cover all cases)

this is link to Chromium sources(FFmpegDemuxer::SeekInternal):

https://chromium.googlesource.com/chromium/src/media/%2B/master/filters/ffmpeg_demuxer.cc

SurpassHR pushed a commit to SurpassHR/ComfyUI-Playground that referenced this pull request Jul 24, 2025
bigcat88 added a commit to bigcat88/ComfyUI that referenced this pull request Jul 25, 2025
rakki194 pushed a commit to rakki194/ComfyUI that referenced this pull request Jul 29, 2025
Vander-Bilt pushed a commit to Vander-Bilt/ComfyUI that referenced this pull request Aug 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants