patchfiles

[GoogleCodeExporter]

Hi Tyro, 

attached you'll find some patchfiles you could have a look at.

Unfortunately, i made some changes before getting your reply about the SVN
patch system, so changes for two different issues are implemented in a
single patch.

-- A first change is the Yahoo download using a csv parser i submitted
before. It seems to work nicely, I haven't seen a failed download yet. As
for the old parser: try downloading symbol "A" for the default timespan.
That gives a parse error on trying to convert a string containing the word
"dividend" to a number.  

For this i included an "Apache 2.0" licensed parser, is that ok?

-- A second change is the addition of a "delete instrument" choice in the
instrument viewer.

small changes, but i'm still in  the process of getting a feel for the
language and the design.

best,

Damiaan

Original issue reported on code.google.com by damiaan....@econ.kuleuven.be on 30 May 2007 at 8:17

Attachments:

• com.ats.resources-patch.txt
• com.ats.model-patch.txt
• com.ats.client-patch.txt

[GoogleCodeExporter]

Deleting an instrument is more complicated than simply deleting the row in the
instrument table.  All of its bar_series and bars must also be deleted, and all 
of
the rows in the stdef_instr_pr must be deleted.  In the UI, at least three 
views must
be refreshed - the instrument view, the available data view, and the strategy 
view.

I see the new CSVReader and the patch deletes YahooHistBuilder, but I don't see 
how
this new class is called.  Can you please attach the modified ImportDataManager 
and
any other class that is necessary to invoke the CSVReader?  

Original comment by tyrotra...@gmail.com on 30 May 2007 at 10:53

• Changed state: Accepted

[GoogleCodeExporter]

Hello Damiaan,
Do you have any new patches?

Original comment by kkazmier...@gmail.com on 16 Jan 2012 at 8:43

[GoogleCodeExporter]

Hi, sorry, no... this was back in 2007, haven't looked at truetrade since.

take care

Original comment by damiaan....@econ.kuleuven.be on 16 Jan 2012 at 9:22