Re: polyfit: add a third output argument, mu [PATCH]

[David Bateman]

Dmitri A. Sergatskov wrote:
> On Mon, Jun 30, 2008 at 1:52 PM, Shaun Jackman <address@hidden> wrote:
>> The following patch adds a third output argument, mu for compatibility with
>> Matlab. I made an effort to update the documentation, but I'm not familiar
>> with Octave's documentation system.
>>
> 
> There is wpolifit in octave-forge that has this and more. And I still
> do not understand
> why it is not in the main octave tree yet...


Because there aren't that many of us who are porting the octave-forge
functions to Octave. I did a bit of this but searched for candidates
based on the function name clashes, and as wpolyfit isn't polyfit I
didn't identify this as a replacement.. I'm sure if you go over wpolyfit
 doing the following

* Add GPL copyright (but keep Paul's public domain copyright)
* Change '' quoted strings to "" quoted strings
* Use parenthesis in if statements
* Write "length (varargin)" and not "length(varargin)" etc
* Other standard Octave style fixes

then John would accept this as a replacement if its clearly better.

D.