octave-maintainers
[Top][All Lists]
Advanced

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

Re: Implementation of importdata belonging to Octave Core or Octave-Forg


From: Juan Pablo Carbajal
Subject: Re: Implementation of importdata belonging to Octave Core or Octave-Forge?
Date: Fri, 19 Oct 2012 23:28:13 +0200

On Fri, Oct 19, 2012 at 7:35 PM, Erik Kjellson
<address@hidden> wrote:
> Hello,
>
> Now I have uploaded my implementation here:
> https://sourceforge.net/p/octave/feature-requests/41/
>
> I've noticed that a lot of function files have some kind of tests in the end
> of the file. I guess that you use them to check that you haven't introduced
> any logic error in the code when modifying it?
> I did find this page: http://wiki.octave.org/Tests
> But I guess I need to fake input from a fake file in some way, is there
> anywhere I can read more on how to do that?
>
> regards,
> Erik
>
>
>
> On 19 October 2012 15:54, Jordi Gutiérrez Hermoso <address@hidden>
> wrote:
>>
>> On 19 October 2012 09:37, Juan Pablo Carbajal <address@hidden>
>> wrote:
>> > On Tue, Oct 16, 2012 at 9:04 PM, Erik Kjellson
>> > <address@hidden> wrote:
>> >> Hello,
>> >>
>> >> I have written an implementation of the Matlab function 'importdata'.
>> >> I'm
>> >> getting a bit confused about the borderline between Octave Core and
>> >> Octave-Forge, so I don't know where it belongs...
>> >>
>> >> My implementation covers most of the functionality in the Matlab
>> >> version,
>> >> but not all. For example, I haven't implemented the automatic detection
>> >> of
>> >> the column separator. Also, for some special cases (like if you in the
>> >> data
>> >> section of the file would have some letters among numeric data) you
>> >> still
>> >> get a slightly different output.
>> >>
>> >> The 'importdata' is, as far as I know, a part of Matlab Core rather
>> >> than a
>> >> toolbox.
>> >> Does the implementation have to be complete to be able to have it in
>> >> Octave
>> >> Core?
>> >>
>> >> Or is it better to add it to the Octave-Forge project for now?
>> >>
>> >> regards,
>> >> Erik
>> >
>> > Hi Erik,
>> >
>> > Cool!
>> >
>> > I would say that if compatibility is not complete then better to keep
>> > it in Forge.
>>
>> Nah, we have tons of incomplete compatibility functions in core. One
>> more will do no harm. It should go in core.
>>
>> > also, you should upload your function seomwhere so we can
>> > see the coding style, Octave core is very strict in this.
>>
>> This is a minor thing. I can adapt its style as necessary.
>>
>> - Jordi G. H.
>
>

You can load a file that will be in the octave folder. Check for
example the demos of imshow
Just choose a small file of your choice and write some test and demos :D

Thank you so much!


reply via email to

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