> From: carandraug+...@gmail.com
> Date: Tue, 2 Oct 2012 11:42:25 +0200
> Subject: Re: [OctDev] Analyze75
> To: adamaitkenh...@hotmail.com
> CC: octave-dev@lists.sourceforge.net
>
> On 2 October 2012 11:11, adam aitkenhead <adamaitkenh...@hotmail.com> wrote:
> >> From: carandraug+...@gmail.com
> >> Date: Tue, 2 Oct 2012 09:33:17 +0200
> >> Subject: Re: [OctDev] Analyze75
> >> To: adamaitkenh...@hotmail.com
> >> CC: octave-dev@lists.sourceforge.net
> >>
> >> On 1 October 2012 22:30, adam aitkenhead <adamaitkenh...@hotmail.com>
> >> wrote:
> >> > I've just submitted analyze75write.m to SVN.
> >>
> >> Nice, thank you. The code looks nice. I have added the function to the
> >> INDEX and NEWS file, and to the see also of the other functions.
> >>
> >> Some tips, you don't need to check the value returned by isfield. If
> >> there is a field with that name, it will return true so "if (isfield
> >> (x, name))" is enough, no need to "if (isfield (x, name) == 1).
> >>
> >> It would be good practice to also have analyze75filename deal with the
> >> filename part, just as it's doing for the other 2 functions. I'd guess
> >> analyze75filename would take an extra argument ("write", "info" or
> >> "read") for the part of checking if the file already exists. This
> >> would also allow to make it return the right error message with:
> >>
> >> error ('analyze75%s: error message.', analyze75function)
> >>
> >> Carnë
> >
> > Those suggestions sound good, I can make those changes although I probably
> > won't have time before the weekend. If you're ready for the release of the
> > image package then don't wait, as it won't affect functionality.
>
> Not yet. There's some warnings and error messages for Mac users that
> should be fixed before.
>
> Carnë
Hi Carnë
I submitted a couple of changes to analyze75read and analyze75write last night
to allow them to work properly for int16 and int32 images. Still to do the
other updates to analyze75write though.
Adam
------------------------------------------------------------------------------
Don't let slow site performance ruin your business. Deploy New Relic APM
Deploy New Relic app performance management and know exactly
what is happening inside your Ruby, Python, PHP, Java, and .NET app
Try New Relic at no cost today and get our sweet Data Nerd shirt too!
http://p.sf.net/sfu/newrelic-dev2dev
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev
