[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Octave-patch-tracker] [patch #8593] Faster version of dlmread(), suppor
From: |
Russell Brinkmann |
Subject: |
[Octave-patch-tracker] [patch #8593] Faster version of dlmread(), supporting escaped delimiters |
Date: |
Tue, 13 Jan 2015 22:46:44 +0000 |
User-agent: |
Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:34.0) Gecko/20100101 Firefox/34.0 |
Follow-up Comment #4, patch #8593 (project octave):
Thanks for the heads up Philip.
I had to spend some time getting octave running again after a new xilinx
install hosed my octave install (fragile library dependancies), so I could run
a quick comparison test.
So just FYI, my code runs significantly slower than csv2cell(). On a test CSV
file I found that csv2cell() parsed in 0.016 seconds and read_dlm_esc() parsed
in 0.292 seconds. However, read_dlm_read() will handle lines with a variable
number of fields, which I required for the project I was working on. So, the
good news is that it wasn't completely wasted effort.
I'll keep my github repo active, just incase anyone finds it useful.
Also a heads up to anyone that uses the read_dlm_esc() code. I took a quick
look at it again, and the starting column has not been implemented yet. I have
a place holder parameter, but no functionality for it.
As for maybe adding to csv2cell(), I'll have to see how my schedule works out
in the next year or so.
Thanks,
Russ
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?8593>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/