Re: [FFmpeg-devel] [PATCH] opus_pvq_search.asm: Handle zero vector input differently.

2017-08-25 Thread Ivan Kalvachev
On 8/25/17, Rostislav Pehlivanov wrote: > On 25 August 2017 at 16:38, Ivan Kalvachev wrote: > >> Instead of returning all zeroes as result and Syy=1.0, >> place all the K pulses in the first element y[0] >> and return Syy=K*K. >> >> This is how the original opus function handles the case. >> This

Re: [FFmpeg-devel] [PATCH] opus_pvq_search.asm: Handle zero vector input differently.

2017-08-25 Thread Ivan Kalvachev
On 8/25/17, Rostislav Pehlivanov wrote: > On 25 August 2017 at 16:38, Ivan Kalvachev wrote: > >> Instead of returning all zeroes as result and Syy=1.0, >> place all the K pulses in the first element y[0] >> and return Syy=K*K. >> >> This is how the original opus function handles the case. >> This

Re: [FFmpeg-devel] [PATCH] opus_pvq_search.asm: Handle zero vector input differently.

2017-08-25 Thread Rostislav Pehlivanov
On 25 August 2017 at 16:38, Ivan Kalvachev wrote: > Instead of returning all zeroes as result and Syy=1.0, > place all the K pulses in the first element y[0] > and return Syy=K*K. > > This is how the original opus function handles the case. > This is how the existing pvq_search_c handles the case

[FFmpeg-devel] [PATCH] opus_pvq_search.asm: Handle zero vector input differently.

2017-08-25 Thread Ivan Kalvachev
Instead of returning all zeroes as result and Syy=1.0, place all the K pulses in the first element y[0] and return Syy=K*K. This is how the original opus function handles the case. This is how the existing pvq_search_c handles the case. Also, according to Rostislav, the encoded all zeros vector w