Displaying 5 results from an estimated 5 matches for "nondiegetic".
2017 Apr 10
2
[Patch] Non-diegetic support for channel mapping 254
...akns for the patch. Here's some comments for now (not done reviewing):
1) You want to use isqrt32() rather than celt_sqrt(), since celt_sqrt()
changes behaviour for fixed-point and provides no guarantee about rounding.
Thanks! I'll make that change.
2) About these two lines:
+ if (nondiegetic_channels && 2 != nondiegetic_channels)
+ return OPUS_UNIMPLEMENTED;
Why not just have the condition be:
+ if (nondiegetic_channels != 2)
We should allow either 0 or 2 nondiegetic channels.
As a minor detail, I prefer N != 2 than 2 != N.
Acknowledged.
Also, unless you t...
2017 Apr 07
2
[Patch] Non-diegetic support for channel mapping 254
Hello all,
Attached is a proposed patch for Opus that allows support for encoding
non-diegetic stereo audio as a coupled stream for use with channel mapping
254. It also rejects channel counts that are not (n+1)^2 + 2j, where n is 0
to 14 and j is either 0 or 1 (See IETF public draft doc attached for
clarification).
Please let me know any suggestions / concerns / comments.
Thank you for your
2017 Apr 24
0
[Patch] Non-diegetic support for channel mapping 254
...ng):
>
> 1) You want to use isqrt32() rather than celt_sqrt(), since celt_sqrt()
> changes behaviour for fixed-point and provides no guarantee about
> rounding.
>
>
> Thanks! I'll make that change.
>
> 2) About these two lines:
> + if (nondiegetic_channels && 2 != nondiegetic_channels)
> + return OPUS_UNIMPLEMENTED;
>
> Why not just have the condition be:
> + if (nondiegetic_channels != 2)
>
>
> We should allow either 0 or 2 nondiegetic channels.
>
>
> As a minor detail...
2017 Apr 25
2
[Patch] Non-diegetic support for channel mapping 254
...rt32() rather than celt_sqrt(), since
> celt_sqrt()
> > changes behaviour for fixed-point and provides no guarantee about
> > rounding.
> >
> >
> > Thanks! I'll make that change.
> >
> > 2) About these two lines:
> > + if (nondiegetic_channels && 2 != nondiegetic_channels)
> > + return OPUS_UNIMPLEMENTED;
> >
> > Why not just have the condition be:
> > + if (nondiegetic_channels != 2)
> >
> >
> > We should allow either 0 or 2 nondiegetic channels.
> &g...
2017 Apr 07
0
[Patch] Non-diegetic support for channel mapping 254
Hi Drew,
Thakns for the patch. Here's some comments for now (not done reviewing):
1) You want to use isqrt32() rather than celt_sqrt(), since celt_sqrt()
changes behaviour for fixed-point and provides no guarantee about rounding.
2) About these two lines:
+ if (nondiegetic_channels && 2 != nondiegetic_channels)
+ return OPUS_UNIMPLEMENTED;
Why not just have the condition be:
+ if (nondiegetic_channels != 2)
As a minor detail, I prefer N != 2 than 2 != N.
Also, unless you think it may one day make sense to have
nondiegetic_channels != 2, I thin...