Bug #72288 for Chart-Strip: Div by zero with

# Tue Nov 08 18:20:20 2011 davidp [...] preshweb.co.uk - Ticket created

Subject:

Div by zero with < 0.5 datapoints

Download (untitled) / with headers
text/plain 330b

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

Download (untitled) / with headers
text/plain 131b

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

Download (untitled) / with headers
text/plain 621b

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

Download chart-strip-test-case.pl
text/x-perl 18k

Message body is not shown because it is too large.

# Thu Nov 10 21:35:20 2011 jaw+pause [...] tcp4me.com - Correspondence added

Download (untitled) / with headers
text/plain 59b

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

Download (untitled) / with headers
text/plain 158b

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'

Bug #72288 for Chart-Strip: Div by zero with < 0.5 datapoints