* Carnë Draug <carandraug+...@gmail.com> [2012-03-17 15:58]:
>
> [snip]
>
> * dataframe - this is a new package and is kinda handy. I think it
> would be good to package it

I am right now setting up the dataframe package for Debian.  I found
two minor issues:

1) There is a wrong path in the inst/dataframe file.  It is fixed with
   the patch attached below.

2) File inst/@dataframe/private/df_name2idx.m is is executable in the
   0.9.1 tarball.  Please, fix it in the next release tarball.

Rafael
Description: Fix path of CSV file needed in the unit testing
Author: Rafael Laboissiere <raf...@laboissiere.net>
Last-Update: 2012-03-22

--- octave-dataframe-0.9.1.orig/inst/dataframe
+++ octave-dataframe-0.9.1/inst/dataframe
@@ -10,7 +10,7 @@
 %!assert(isempty(x.rowidx), false);
 %! x.types(2) ='single';
 %!assert(class(x.array(1, 2)), 'single')
-%! x=dataframe('octave_frame/data_test.csv');
+%! x=dataframe('data_test.csv');
 %!assert(isna(x.array(9, 4)))
 %! # remove rownames
 %! x.rownames = [];
------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here 
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev

Reply via email to