<br><br><div class="gmail_quote">On Sun, Jul 29, 2012 at 2:23 PM, Thomas Haslwanter <span dir="ltr">&lt;<a href="mailto:thomas.haslwanter@alumni.ethz.ch" target="_blank">thomas.haslwanter@alumni.ethz.ch</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
I found an error on the SciPy Cookbook (described below), and wanted to fix it.<br>
So I wrote a corrected version and a unittest procedure. But now I don&#39;t know<br>
how I can submit/contribute it to SciPy. (BTW, the wiki did not let me to change<br>
it, even after I registered.) </blockquote><div><br>That&#39;s odd. Can other people edit that page (I can)?<br> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Is the only way to pull the whole sourcecode,<br>
install Fortran and C++ compilers, etc? Or can I just submit Python code?<br>
Please let me know.<br>
<br></blockquote><div>The Cookbook isn&#39;t part of the SciPy source code, so in this case the only way is to edit the wiki page. Or open a ticket as you did: <a href="http://projects.scipy.org/scipy/ticket/1706">http://projects.scipy.org/scipy/ticket/1706</a><br>
<br>If your question is how you would go about adding this function in scipy.signal, which as I noted on #1706 was discussed before, you can attach patches to Trac. The better way (i.e. lower overhead for scipy devs) would indeed be to use git and send a pull request. How to do that is described at <a href="https://github.com/scipy/scipy/blob/master/HACKING.rst.txt">https://github.com/scipy/scipy/blob/master/HACKING.rst.txt</a>.<br>
<br>Ralf<br><br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The problem:<br>
 The Cookbook entry<br>
 <a href="http://www.scipy.org/Cookbook/SavitzkyGolay" target="_blank">http://www.scipy.org/Cookbook/SavitzkyGolay</a><br>
 has a number of problems:<br>
<br>
 1) As it is, it does in general not run. The line<br>
    &quot;import numpy as np&quot;<br>
 should be inserted before the first &quot;try&quot; in line 49<br>
<br>
 2) I don&#39;t completely understand the workings of the code. However, when<br>
 you put in a sine-wave and calculate the first derivative,     you get out<br>
 a negative(!) cosine - not a cosine. The correct result appears if you<br>
 replace &quot;m,&quot; in line 68 with &quot;m[::-1],&quot;<br>
<br>
 Since the Savitzky-Golay filter is a very commonly used filter, I regard<br>
 this mistake as significant.<br>
<br>
thomas<br>
<br>
_______________________________________________<br>
SciPy-Dev mailing list<br>
<a href="mailto:SciPy-Dev@scipy.org">SciPy-Dev@scipy.org</a><br>
<a href="http://mail.scipy.org/mailman/listinfo/scipy-dev" target="_blank">http://mail.scipy.org/mailman/listinfo/scipy-dev</a><br>
</blockquote></div><br>