search for: analysis_frame_size

...4c, analysis_size=analysis_size at entry=960, c1=c1 at entry=0, c2=c2 at entry=-2, analysis_channels=1, downmix=downmix at entry=0xa80fe0 <downmix_int>, float_api=float_api at entry=0) at src/opus_encoder.c:1826 #5 0x0000000000a85211 in opus_encode (st=0x158935a0, pcm=0x7f26b5a1084c, analysis_frame_size=960, data=0x7f26b5a1084c "", max_data_bytes=5988) at src/opus_encoder.c:2227 #6 0x00000000004ce892 in opus_encoder::transcode (this=0x21a30200, in_packet=0x7f267402cf30) at /root/mediagateway/source/engine/media-objects/transcoder/codecs/opus.h:91 (gdb) frame 0 #0 0x000000000...

On Wed, Jun 4, 2014 at 4:31 PM, Timothy B. Terriberry <tterribe at xiph.org> wrote: > Alpha Thinktink wrote: >> max_data_bytes=-11 > > That value is passed in by you. I also don't think passing such a value > would have worked in earlier releases. It indicates the size of the > buffer you are passing to the encoder to receive the encoded output. Actually, I'm

...960, c1=c1 at entry=0, >> c2=c2 at entry=-2, analysis_channels=1, >> downmix=downmix at entry=0xa80fe0 <downmix_int>, >> float_api=float_api at entry=0) at src/opus_encoder.c:1826 >> #5 0x0000000000a85211 in opus_encode (st=0x158935a0, >> pcm=0x7f26b5a1084c, analysis_frame_size=960, >> data=0x7f26b5a1084c "", max_data_bytes=5988) at src/opus_encoder.c:2227 >> #6 0x00000000004ce892 in opus_encoder::transcode (this=0x21a30200, >> in_packet=0x7f267402cf30) >> at >> /root/mediagateway/source/engine/media-objects/transcoder/code...

...and s was 0. then line 849 curr_max = IMIN(curr_max,MS_FRAME_TMP); which left curr_max at -13 The result at line 853 is opus_encode_native is called with parameters so thusly: (encoder state pointer, buf pointer, frame_size=960, tmp_data pointer, curr_max=-13, lsb_depth=24, pcm pointer, analysis_frame_size=960, c1=0, c2=-1, st->layout.nb_channels=4, downmix pointer, float_api=1); I don't know what to suggest as I've never tinkered with the opus library internals, nor have I ever compiled it before today and do not know what it looked like in previous versions. Sorry.

...curr_max = IMIN(curr_max,MS_FRAME_TMP); > > which left curr_max at -13 > > The result at line 853 is opus_encode_native is called with parameters > so thusly: > (encoder state pointer, buf pointer, frame_size=960, tmp_data pointer, > curr_max=-13, lsb_depth=24, pcm pointer, analysis_frame_size=960, > c1=0, c2=-1, st->layout.nb_channels=4, downmix pointer, float_api=1); > > > I don't know what to suggest as I've never tinkered with the opus > library internals, nor have I ever compiled it before today and do not > know what it looked like in previous version...

...lysis_size at entry=960, c1=c1 at entry=0, > c2=c2 at entry=-2, analysis_channels=1, >     downmix=downmix at entry=0xa80fe0 <downmix_int>, > float_api=float_api at entry=0) at src/opus_encoder.c:1826 > #5  0x0000000000a85211 in opus_encode (st=0x158935a0, > pcm=0x7f26b5a1084c, analysis_frame_size=960, >     data=0x7f26b5a1084c "", max_data_bytes=5988) at src/opus_encoder.c:2227 > #6  0x00000000004ce892 in opus_encoder::transcode (this=0x21a30200, > in_packet=0x7f267402cf30) >     at > /root/mediagateway/source/engine/media-objects/transcoder/codecs/opus.h:91 > &...

...so we only need to check one value. */ if (celt_isnan(out[0].r)) @@ -635,7 +635,7 @@ static void tonality_analysis(TonalityAnalysisState *tonal, const CELTMode *celt void run_analysis(TonalityAnalysisState *analysis, const CELTMode *celt_mode, const void *analysis_pcm, int analysis_frame_size, int frame_size, int c1, int c2, int C, opus_int32 Fs, - int lsb_depth, downmix_func downmix, AnalysisInfo *analysis_info) + int lsb_depth, downmix_func downmix, AnalysisInfo *analysis_info, int arch) { int offset; int pcm_len; @@ -648,7 +648,7 @@ void run_a...

...so we only need to check one value. */ if (celt_isnan(out[0].r)) @@ -635,7 +635,7 @@ static void tonality_analysis(TonalityAnalysisState *tonal, const CELTMode *celt void run_analysis(TonalityAnalysisState *analysis, const CELTMode *celt_mode, const void *analysis_pcm, int analysis_frame_size, int frame_size, int c1, int c2, int C, opus_int32 Fs, - int lsb_depth, downmix_func downmix, AnalysisInfo *analysis_info) + int lsb_depth, downmix_func downmix, AnalysisInfo *analysis_info, int arch) { int offset; int pcm_len; @@ -648,7 +648,7 @@ void run_a...

...so we only need to check one value. */ if (celt_isnan(out[0].r)) @@ -635,7 +635,7 @@ static void tonality_analysis(TonalityAnalysisState *tonal, const CELTMode *celt void run_analysis(TonalityAnalysisState *analysis, const CELTMode *celt_mode, const void *analysis_pcm, int analysis_frame_size, int frame_size, int c1, int c2, int C, opus_int32 Fs, - int lsb_depth, downmix_func downmix, AnalysisInfo *analysis_info) + int lsb_depth, downmix_func downmix, AnalysisInfo *analysis_info, int arch) { int offset; int pcm_len; @@ -648,7 +648,7 @@ void run_a...

Changes from RFC PATCH v3 - Just rebased on tip - For all else, please see notes from RFC PATCH v3 at http://lists.xiph.org/pipermail/opus/2015-March/002902.html - latest wip opus tree/branch https://git.linaro.org/people/viswanath.puttagunta/Ne10.git branch: rfcv4_final_fft_ne10 Viswanath Puttagunta (1): armv7(float): Optimize encode usecase using NE10 library Makefile.am

Changes from RFC PATCH v1: - passing arch parameter explicitly - reduced stack usage by ~3.5K by using scaled NE10 fft version - moved all optimization array functions to arm_celt_map.c - Other cleanups pointed out by Timothy Phil, As you mentioned earlier, could you please address all compile and linker errors/warnings coming out of Ne10 library? You can find my working Ne10 repo at [1] You

Changes from RFC PATCH v2 - fixed compile issue when just compiling for --enable-intrinsics for ARMv7 without NE10 - Notes for NE10: - All compile/link warnings are now in upstream NE10 - Only patch pending upstream in NE10 is the one that needs to add -funsafe-math-optimizations for ARMv7 targets. - Phil Wang @ ARM is working on getting this fixed. - Note that even without

Hi All, Since I continue to base my work on top of Jonathan's patch, and my previous Ne10 fft/ifft/mdct_forward/backward patches, I thought it would be better to just post all new patches as a patch series. Please let me know if anyone disagrees with this approach. You can see wip branch of all latest patches at https://git.linaro.org/people/viswanath.puttagunta/opus.git Branch:

Hi Timothy, As I mentioned earlier [1], I now fixed compile issues with fixed point and resubmitting the patch. I also have new patch that does intrinsics optimizations for celt_pitch_xcorr targetting aarch64. You can find my latest work-in-progress branch at [2] For reference, you can use the Ne10 pre-built libraries at [3] Note that I am working with Phil at ARM to get my patch at [4]

Hi All, As per Timothy's suggestion, disabling mdct_forward for fixed point. Only effects armv7,armv8: Extend fixed fft NE10 optimizations to mdct Rest of patches are same as in [1] For reference, latest wip code for opus is at [2] Still working with NE10 team at ARM to get corner cases of mdct_forward. Will update with another patch when issue in NE10 gets fixed. Regards, Vish [1]:

Hi All, Changes from RFC v2 [1] armv7,armv8: Extend fixed fft NE10 optimizations to mdct - Overflow issue fixed by Phil at ARM. Ne10 wip at [2]. Should be upstream soon. - So, re-enabled using fixed fft for mdct_forward which was disabled in RFCv2 armv7,armv8: Optimize fixed point fft using NE10 library - Thanks to Jonathan Lennox, fixed some build fixes on iOS and some copy-paste errors Rest

Hello Timothy / Jean-Marc / opus-dev, This patch series is follow up on work I posted on [1]. In addition to what was posted on [1], this patch series mainly integrates Fixed point FFT implementations in NE10 library into opus. You can view my opus wip code at [2]. Note that while I found some issues both with the NE10 library(fixed fft) and with Linaro toolchain (armv8 intrinsics), the work