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

Ben Abbott bpabbott at mac.com
Tue Jul 1 06:02:12 CDT 2008


On Jul 1, 2008, at 2:23 AM, David Bateman wrote:

> Dmitri A. Sergatskov wrote:
>> On Mon, Jun 30, 2008 at 1:52 PM, Shaun Jackman <sjackman at bcgsc.ca>  
>> 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.

Hey guys,

A changeset was applied back in Feb that includes the third output  
argument, mu.

	http://www.nabble.com/Re%3A--changeset--Polyfit-with-scaling-p15834853.html

I checked my copy of jwe's default archive, and it is present there.  
Is it not in yours?

Ben




More information about the Octave-maintainers mailing list