<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://www.w3.org/Bugs/Public/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4"
          urlbase="https://www.w3.org/Bugs/Public/"
          
          maintainer="sysbot+bugzilla@w3.org"
>

    <bug>
          <bug_id>17403</bug_id>
          
          <creation_ts>2012-06-05 12:28:55 +0000</creation_ts>
          <short_desc>(LinearConvolution): Linear convolution effect is defined by Wikipedia</short_desc>
          <delta_ts>2014-10-28 17:16:39 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>AudioWG</product>
          <component>Web Audio API</component>
          <version>unspecified</version>
          <rep_platform>PC</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>CLOSED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>TBD</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael[tm] Smith">mike</reporter>
          <assigned_to name="Chris Rogers">crogers</assigned_to>
          <cc>mage</cc>
    
    <cc>ot</cc>
    
    <cc>public-audio</cc>
          
          <qa_contact name="This bug has no owner yet - up for the taking">dave.null</qa_contact>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>68715</commentid>
    <comment_count>0</comment_count>
    <who name="Michael[tm] Smith">mike</who>
    <bug_when>2012-06-05 12:28:55 +0000</bug_when>
    <thetext>Audio-ISSUE-69 (LinearConvolution): Linear convolution effect is defined by Wikipedia [Web Audio API]

http://www.w3.org/2011/audio/track/issues/69

Raised by: Philip Jägenstedt
On product: Web Audio API

https://dvcs.w3.org/hg/audio/raw-file/tip/webaudio/specification.html#ConvolverNode

links to

https://dvcs.w3.org/hg/audio/raw-file/tip/webaudio/specification.html#Convolution-section

links to

http://en.wikipedia.org/wiki/Convolution

The definition should be in the specification itself. A problem that is not addressed at all is how phase/latency depends on the length of the convolution buffer. In other words, it must be defined exactly how each sample value is calculated from the the input signal and the impulse response buffer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68784</commentid>
    <comment_count>1</comment_count>
    <who name="Olivier Thereaux">olivier.thereaux</who>
    <bug_when>2012-06-07 08:17:23 +0000</bug_when>
    <thetext>[admin] Assigning items currently being worked on by editor.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68891</commentid>
    <comment_count>2</comment_count>
    <who name="Chris Rogers">crogers</who>
    <bug_when>2012-06-09 00:06:34 +0000</bug_when>
    <thetext>I believe the operation &quot;linear convolution&quot; is a sufficiently well-known and well-defined operation in mathematics that no further explanation is required, anymore than the definition of a cos() function. It is defined in many texts on math/DSP.

I will add an &quot;informative&quot; link to a page which can help guide implementors to practically implement this well-defined mathematical operation. I am happy also to point to working, cross-platform, well-optimized code having a non-restrictive license. I can also point to an alternate implementation &quot;Freeverb&quot;:
http://freeverb3.sourceforge.net/

FIXED:
https://dvcs.w3.org/hg/audio/rev/7865f8b76f19</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>68952</commentid>
    <comment_count>3</comment_count>
    <who name="Marcus Geelnard (Opera)">mage</who>
    <bug_when>2012-06-12 08:43:52 +0000</bug_when>
    <thetext>I think this mail-comment from the original issue was overlooked:

&quot;
The new document [1] contains valuable information, but isn&apos;t what this  
issue was about. If linear convolution need not be defined, then the issue  
should be closed as invalid.

Not everyone reading the spec will have a background in signal processing,  
e.g. we want our QA to be able to write tests with little effort. We  
suggest to define the output y in terms of the input x and the [possibly normalized] impulse  
response g as a summation formula or pseudo-code such as:

y[n] = 0
for i in 0...N-1:
   y[n] += x[n-i] * g[i]

This leaves no room for ambiguity about the phase of the output signal.

ISSUE-70 will need to be resolved for the above formula to make sense,  
though, as the input and impulse response may have different numbers of  
channels.

[1] https://dvcs.w3.org/hg/audio/raw-file/tip/webaudio/convolution.html
&quot;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>113867</commentid>
    <comment_count>4</comment_count>
    <who name="Olivier Thereaux">olivier.thereaux</who>
    <bug_when>2014-10-28 17:13:47 +0000</bug_when>
    <thetext>Web Audio API issues have been migrated to Github. 
See https://github.com/WebAudio/web-audio-api/issues</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>114010</commentid>
    <comment_count>5</comment_count>
    <who name="Olivier Thereaux">olivier.thereaux</who>
    <bug_when>2014-10-28 17:16:39 +0000</bug_when>
    <thetext>Closing. See https://github.com/WebAudio/web-audio-api/issues for up to date list of issues for the Web Audio API.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>