mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2025-10-29 05:40:27 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

audioconvert: resampler: fix off-by-one issues

Browse source 
Resampler without prefill was sometime outputting with different delay
than with prefill. Adjust initial history by 1 which seems to bring it
more in line.

The resampler phase also appears to depend on how many samples remain in
history which leads to possibly unexpected +-1 variation. Take this into
account in reported phase.

These changes make the resampler delay tests pass.

Both changes are sort of empirical --- I don't fully understand why
these would fix things but they seem to be needed to make the delay
calculations agree with what the resampler outputs.
This commit is contained in:
Pauli Virtanen 2025-01-16 19:40:30 +02:00
parent 79384530b5
commit eb91f097d9
1 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
spa/plugins/audioconvert/resample-native.c
View file

@ -305,7 +305,7 @@ static void impl_native_reset (struct resample *r)
if (r->options & RESAMPLE_OPTION_PREFILL)
d->hist = d->n_taps - 1;
else
d->hist = (d->n_taps / 2) - 1;
d->hist = d->n_taps / 2;
d->phase = 0;
}
@ -322,14 +322,18 @@ static int32_t impl_native_phase_ns(struct resample *r)
if (d->func == d->info->process_full) {
float pho = -(float)((int32_t)d->phase) / d->out_rate;
if (pho != 0.0f)
/* XXX: this is how it seems to behave, but not clear why */
if (d->hist >= d->n_taps - 1)
pho += 1.0f;
return (int32_t)(pho * SPA_NSEC_PER_SEC / r->i_rate);
} else if (d->func == d->info->process_inter) {
float pho = -d->phase / d->out_rate;
if (pho != 0.0f)
/* XXX: this is how it seems to behave, but not clear why */
if (d->hist >= d->n_taps - 1)
pho += 1.0f;
return (int32_t)(pho * SPA_NSEC_PER_SEC / r->i_rate);
}