search for: celtmod

...------------ libcelt/bands.h | 6 ++---- libcelt/celt.c | 6 ++---- 3 files changed, 28 insertions(+), 27 deletions(-) diff --git a/libcelt/bands.c b/libcelt/bands.c index d0c3498..315cce8 100644 --- a/libcelt/bands.c +++ b/libcelt/bands.c @@ -247,23 +247,6 @@ void compute_pitch_gain(const CELTMode *m, const celt_norm_t *X, const celt_norm }*/ } -/* Apply the (quantised) gain to each "pitch band" */ -void pitch_quant_bands(const CELTMode *m, celt_norm_t * restrict P, const celt_pgain_t * restrict gains) -{ - int i; - const celt_int16_t *pBands = m->pBands; - const i...

Hi I noticed a crash issue when I passed the following values: celt_mode_create(96000, 258, &e); CELTMode->mdct.kfft[0] is not initialized after calling?clt_mdct_init() and when?celt_mode_destroy() is called it tries to dereference this value in kiss_fft_free(). -- Bjoern Here's the callstack: !kiss_fft_free(const kiss_fft_state * cfg=0x00000000) ?Line 650 + 0x3 bytes C!clt_mdct_clear(mdct_look...

...itch.h psy.h \ + quant_bands.h rate.h stack_alloc.h vq.h noinst_PROGRAMS = testcelt dump_modes testcelt_SOURCES = testcelt.c diff --git a/libcelt/bands.c b/libcelt/bands.c index 315cce8..49eaeda 100644 --- a/libcelt/bands.c +++ b/libcelt/bands.c @@ -210,9 +210,10 @@ void denormalise_bands(const CELTMode *m, const celt_norm_t * restrict X, celt_s /* Compute the best gain for each "pitch band" */ -void compute_pitch_gain(const CELTMode *m, const celt_norm_t *X, const celt_norm_t *P, celt_pgain_t *gains) +int compute_pitch_gain(const CELTMode *m, const celt_norm_t *X, const celt_norm_...

....c | 14 +++++++------- libcelt/quant_bands.h | 2 +- 15 files changed, 83 insertions(+), 84 deletions(-) diff --git a/libcelt/bands.c b/libcelt/bands.c index 1948bae..d51feee 100644 --- a/libcelt/bands.c +++ b/libcelt/bands.c @@ -1161,7 +1161,7 @@ static unsigned quant_band(int encode, const CELTMode *m, int i, celt_norm *X, c void quant_all_bands(int encode, const CELTMode *m, int start, int end, celt_norm *_X, celt_norm *_Y, unsigned char *collapse_masks, const celt_ener *bandE, int *pulses, int shortBlocks, int spread, int dual_stereo, int intensity, int *tf_res, int resynth,...

...-) diff --git a/celt/bands.c b/celt/bands.c index bbe8a4c..1ab24aa 100644 --- a/celt/bands.c +++ b/celt/bands.c @@ -92,10 +92,11 @@ static int bitexact_log2tan(int isin,int icos) #ifdef FIXED_POINT /* Compute the amplitude (sqrt energy) in each of the bands */ -void compute_band_energies(const CELTMode *m, const celt_sig *X, celt_ener *bandE, int end, int C, int LM) +void compute_band_energies(const CELTMode *m, const celt_sig *X, celt_ener *bandE, int end, int C, int LM, int arch) { int i, c, N; const opus_int16 *eBands = m->eBands; + (void)arch; N = m->shortMdctSize<&lt...

...ter to celt_encoder_create() is that celt_encoder_init() doesn't free the CELTEncoder ('st' parameter) when 'if (channels < 0 || channels > 2)' evaluates to true. Seems celt_encoder_init() should do like this instead: CELTEncoder *celt_encoder_init(CELTEncoder *st, const CELTMode *mode, int channels, int *error) { if (channels < 0 || channels > 2) { if (error) *error = CELT_BAD_ARG; celt_free(st); /* ensure 'st' is deallocated */ return NULL; } Or maybe even better have celt_encoder_init() allocate the CELTEncoder so it...

...a3fbb..7a2c71b 100644 --- a/celt/celt_encoder.c +++ b/celt/celt_encoder.c @@ -414,7 +414,8 @@ int patch_transient_decision(opus_val16 *newE, opus_val16 *oldE, int nbEBands, /** Apply window and compute the MDCT for all sub-frames and all channels in a frame */ static void compute_mdcts(const CELTMode *mode, int shortBlocks, celt_sig * OPUS_RESTRICT in, - celt_sig * OPUS_RESTRICT out, int C, int CC, int LM, int upsample) + celt_sig * OPUS_RESTRICT out, int C, int CC, int LM, int upsample, + int arch) { const int ove...

...a3fbb..7a2c71b 100644 --- a/celt/celt_encoder.c +++ b/celt/celt_encoder.c @@ -414,7 +414,8 @@ int patch_transient_decision(opus_val16 *newE, opus_val16 *oldE, int nbEBands, /** Apply window and compute the MDCT for all sub-frames and all channels in a frame */ static void compute_mdcts(const CELTMode *mode, int shortBlocks, celt_sig * OPUS_RESTRICT in, - celt_sig * OPUS_RESTRICT out, int C, int CC, int LM, int upsample) + celt_sig * OPUS_RESTRICT out, int C, int CC, int LM, int upsample, + int arch) { const int ove...

...a3fbb..7a2c71b 100644 --- a/celt/celt_encoder.c +++ b/celt/celt_encoder.c @@ -414,7 +414,8 @@ int patch_transient_decision(opus_val16 *newE, opus_val16 *oldE, int nbEBands, /** Apply window and compute the MDCT for all sub-frames and all channels in a frame */ static void compute_mdcts(const CELTMode *mode, int shortBlocks, celt_sig * OPUS_RESTRICT in, - celt_sig * OPUS_RESTRICT out, int C, int CC, int LM, int upsample) + celt_sig * OPUS_RESTRICT out, int C, int CC, int LM, int upsample, + int arch) { const int ove...

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

...for writing\n",outputfilename); return -1; } int bytes_per_packet = ((gBitRate*gFrameSize/gSampleRate+4)/8)*gNrChannels; int error=0; celt_int16* pcmbuffer=(celt_int16*)malloc(sizeof(celt_int16)*gFrameSize*gNrChannels); unsigned char* encodedbuffer=(unsigned char*)malloc(bytes_per_packet); CELTMode* mode=celt_mode_create(gSampleRate,gFrameSize,&error); if(mode==NULL || error!=CELT_OK) { printf("ERROR: celt_mode_create %s\n",celt_strerror(error)); return -1; } int offset=0; int res=0; CELTEncoder* encoder=celt_encoder_create(mode,gNrChannels,&error); if(encoder==NULL ||...

...embers (via pitch_state_alloc and quant_prob_alloc respectively). I also notice that in dump_modes.c, these two members are just set to 0 explicitly. Why aren't these two members part of the static mode? Would anything bad happen if I make them part of the static mode? I assume not because CELTMode is read-only right? Thanks for your help. -- Chen-Po Sun Firelight Technologies http://www.fmod.org/

...ereo channel with Fs=48000 and 128 byte frames, but get NULL returned from the celt_mode_create function. I am not defining CUSTOM_MODES_ONLY (because I don't know what it means). On the last couple of lines of code, there is no error (error=0), but it appears that you always return NULL..? CELTMode *celt_mode_create(celt_int32 Fs, int frame_size, int *error) #ifndef CUSTOM_MODES_ONLY for (i=0;i<TOTAL_MODES;i++) { int j; for (j=0;j<4;j++) { if (Fs == static_mode_list[i]->Fs && (frame_size<<j) == static_mode_list...

Hello opus-dev, I've been cooking up this patchset to integrate NE10 library into opus. Current patchset focuses on encode use case mainly effecting performance of clt_mdct_forward() and opus_fft() (for float only) Glad to report the following on Encode use case: (Measured on my Beaglebone Black Cortex-A8 board) - Performance improvement for encode use case ~= 12.34% (Based on time -p

...) +#endif /* OPUS_HAVE_RTCD */ +#endif /* !defined(FIXED_POINT) && defined(HAVE_ARM_NE10) */ + +#endif diff --git a/celt/celt_encoder.c b/celt/celt_encoder.c index 86a3fbb..9de9a92 100644 --- a/celt/celt_encoder.c +++ b/celt/celt_encoder.c @@ -435,7 +435,9 @@ static void compute_mdcts(const CELTMode *mode, int shortBlocks, celt_sig * OPUS for (b=0;b<B;b++) { /* Interleaving the sub-frames while doing the MDCTs */ - clt_mdct_forward(&mode->mdct, in+c*(B*N+overlap)+b*N, &out[b+c*N*B], mode->window, overlap, shift, B); + clt_mdct_forward(&...

...src/opus_multistream_encoder.c +++ b/src/opus_multistream_encoder.c @@ -454,7 +454,9 @@ static int opus_multistream_encode_native unsigned char tmp_data[MS_FRAME_TMP]; OpusRepacketizer rp; opus_int32 complexity; +#ifndef FIXED_POINT AnalysisInfo analysis_info; +#endif const CELTMode *celt_mode; opus_int32 bitrates[256]; opus_val16 bandLogE[42]; @@ -629,27 +631,6 @@ static int opus_multistream_encode_native } -#if !defined(DISABLE_FLOAT_API) -static void opus_copy_channel_in_float( - opus_val16 *dst, - int dst_stride, - const void *src, - int src_stride, - i...

...lt/dump_modes/dump_modes_arch.h index 1436926..59073ee 100644 --- a/celt/dump_modes/dump_modes_arch.h +++ b/celt/dump_modes/dump_modes_arch.h @@ -28,11 +28,17 @@ #ifndef DUMP_MODE_ARCH_H #define DUMP_MODE_ARCH_H +#include "config.h" + void dump_modes_arch_init(); void dump_mode_arch(CELTMode *mode); void dump_modes_arch_finalize(); +#if !defined(FIXED_POINT) #define ARM_NE10_ARCH_FILE_NAME "static_modes_float_arm_ne10.h" +#else +#define ARM_NE10_ARCH_FILE_NAME "static_modes_fixed_arm_ne10.h" +#endif #if defined(HAVE_ARM_NE10) #define OVERRIDE_FFT (1) diff -...

...coding/neteq4/audio_classifier.h (part of the above patch) refers to following internal headers: #include "third_party/opus/src/celt/celt.h" #include "third_party/opus/src/src/analysis.h" #include "third_party/opus/src/src/opus_private.h" I can see references to e.g. CELTMode in that webrtc file. Do you have recommendations how WebRTC project could use a public Opus API for this? Please let me know if you have any questions. This is my first thread I post to this ML so I apologize for any newbie mistakes. I've tried to read some docs on Opus homepage and have bee...