Re: [GT] scan.pl error

["Todd Turner" <todd AT turnernet.org>]

Hi Robert,

Your modified scan.pl works.  Thanks!

Todd

On Fri, Mar 21, 2008 at 8:45 PM, Robert A. Schmied <ras
AT
acm.org> wrote:
> Robert A. Schmied wrote:
>  > Todd Turner wrote:
>  >
>  >> I'm seeing the same behavior on revision 545 (mar 1).
>  >> display_indicator and display_signal both are working as expected.
>  >
>  >
>  > interesting ...
>  >
>  >>
>  >> Looking at the error message:
>  >>
>  >> Argument "day" isn't numeric in numeric gt (>) at ../GT/Tools.pm line
>  >> 453, <> line 1.
>  >> Can't call method "timeframe" on an undefined value at
>  >> ../GT/Calculator.pm line 70, <> line 1.
>  >>
>  >> Corresponds to the if statement in this codeblock:
>  >>
>  >> foreach(reverse(@tf)) {
>  >>   next if ($_ > $timeframe);
>  >>   $q = $db->get_last_prices($code, $max_loaded_items, $_);
>  >>   last if ($q->count > 0);
>  >> }
>  >
>  >
>  > humm "CLOSE" isn't a timeframe reference of any sort,
>  > and "day" is the timeframe name. which seems wrong too
>  > since the '>' is looking for the timeframe enumeration (70 iirc)
>  > not the name.
>  >
>  > i've got to do a thing, but will look at this in more detail later
>  >
>  > ras
>  >
>  >
>
>  oops -- seemed to have missed a chunk of code. try the attached scan.pl.
>  if it works let list know and someone will commit it in due course ...
>
>  this file it has these changes:
>    i) adds the missing pieces of code
>   ii) tweaks a trailing blank or two; changes empty string to '' from ""
>   iii) changes the text format separator to ' - ' (space hyphen space)
>       from '\t ' (tab space) between code and company name in output report
>   iv) renames internal sub local_parse_date_str from parse_date_str.
>       this sub could be removed (and will likely be when script committed)
>
>  please undo any of the previous undoing and use the most recent gt files
>
>  sorry for all the trouble
>
>
>  ras
>
>