octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: be registered as an Octave Forge developer


From: Mike Miller
Subject: Re: be registered as an Octave Forge developer
Date: Thu, 20 Mar 2014 08:29:02 -0400
User-agent: Mutt/1.5.22 (2013-10-16)

On Thu, Mar 20, 2014 at 10:41:01 +0100, Bastian Ebeling wrote:
> Hi there,
> 
> this is me Bastian (barry1 on sf.net).
> 
> I'd like to aplly a change to
> octave/packages/signal-1.1.3/czt.m
> 
> given as this diff
> 
> --- /usr/share/octave/packages/signal-1.1.3/czt.m 2012-05-17
> 22:38:46.000000000 +0200
> +++ czt.m 2014-03-20 08:16:59.257637103 +0100
> @@ -65,6 +65,8 @@
>    nfft = 2^nextpow2(n+m-1); # fft pad
>    W2 = w.^(([-(n-1):max(m-1,n-1)]'.^2)/2); # chirp
> 
> +  y=zeros(row,col); # y should not grow in loop
> +
>    for idx = 1:col
>      fg = fft(x(:,idx).*(a.^-(N-n)).*W2(N), nfft);
>      fw = fft(1./W2(NM), nfft);
> 
> 
> You can see, the only change is a preallocation command for the output y
> the prepend growing in the loop.

This change looks reasonable, aside from the coding style. Thanks for
offering to help.

Please try to follow the Octave guidelines for this and future
contributions:
  
https://www.gnu.org/software/octave/doc/interpreter/Contributing-Guidelines.html

I am not the right person to answer about adding you to Octave Forge,
but now that the signal package is using a DVCS, you have many more
options.

You can "hg export" your changeset as a patch and post it to the Octave
patch tracker:
  https://savannah.gnu.org/patch/?func=additem&group=octave

You can push your changes to another hg repository that you own and host
publically, and ask (on the patch tracker) for the change to be pulled.

You can "fork" my copy of the signal repository on bitbucket and issue a
pull request using bitbucket's tools:
  https://bitbucket.org/mtmiller/octave-signal

FYI, I push changes more frequently to the following repository, simply
because it's mine and it's faster for me than sourceforge, so it may be
ahead of what's on sourceforge at times:
  https://hg.mtmxr.com/octave-signal/

Thanks again, I'm looking forward to your future contributions.

-- 
mike

Attachment: signature.asc
Description: Digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]