Skip Menu |

This queue is for tickets about the Chart-Strip CPAN distribution.

Report information
The Basics
Id: 72288
Status: resolved
Priority: 0/
Queue: Chart-Strip
People
Owner: Nobody in particular
Requestors: davidp [...] preshweb.co.uk
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 1.07
Fixed in: (no value)

History Show all quoted text
  Tue Nov 08 18:20:20 2011 davidp [...] preshweb.co.uk - Ticket created
Subject: Div by zero with < 0.5 datapoints
Trying to chart a dataset with values below 0.5, it seems the data points are being rounded down to zero then used in division at line 700. Illegal division by zero at /usr/local/share/perl/5.10.0/Chart/Strip.pm line 700. 699: $st = floor( $tp / $is ) * $is; # -> 4 - 8, ceil -> 2 - 4 700: $low = int( $min / $st ) * $st;
  Tue Nov 08 19:19:31 2011 jaw+pause [...] tcp4me.com - Correspondence added
module has been tested with datapoints less than 0.5. cannot reproduce. please include sample code that demonstrates the problem.
  Tue Nov 08 19:19:32 2011 The RT System itself - Status changed from 'new' to 'open'
  Tue Nov 08 19:42:15 2011 davidp [...] preshweb.co.uk - Correspondence added
Thanks for the fast reply! On 2011-11-09 00:19:31, JAW wrote: Show quoted text
> module has been tested with datapoints less than 0.5. > cannot reproduce. > > please include sample code that demonstrates the problem.
Ah - in knocking up a test case to produce, I've found it only seems to happen with lots of data points. The test case I used is attached; it generates a graph fine all the way up until it reaches the attempt with 89 data points, then crashes: [...] Testing with 87 data points Testing with 88 data points Testing with 89 data points Illegal division by zero at /usr/local/share/perl/5.10.0/Chart/Strip.pm line 700.
Subject: chart-strip-test-case.pl

Message body is not shown because it is too large.

  Thu Nov 10 21:35:20 2011 jaw+pause [...] tcp4me.com - Correspondence added
fixed. new version should show up on cpan shortly. thanks.
  Thu Nov 10 21:35:21 2011 jaw+pause [...] tcp4me.com - Status changed from 'open' to 'resolved'
  Fri Nov 11 05:40:38 2011 davidp [...] preshweb.co.uk - Correspondence added
Great stuff, thanks! Just tried it out and works a treat now for all the periods I was trying to graph, even when a massive amount of datapoints are given :)
  Fri Nov 11 05:40:40 2011 The RT System itself - Status changed from 'resolved' to 'open'
  Tue Nov 15 10:51:47 2011 jaw+pause [...] tcp4me.com - Status changed from 'open' to 'resolved'