jhinkel / SenseMaking

0 stars 0 forks source link

FrequencyByMonth array seems additive rather than per month #1

Closed werdnanoslen closed 7 years ago

werdnanoslen commented 9 years ago

FrequencyByMonth should return an array of 12 elements, one frequency for each month. However, it seems the function is returning an array where each element is that month plus the previous month. That is, I'd expect to see something like [0, 0, 3, 2, 0, 5, 0, 1, 0, 0, 1, 2], but instead it seems to return something like [0, 0, 3, 5, 5, 10, 10, 11, 11, 11, 12, 14]. (Compare the pattern to the first example array).

jhinkel commented 9 years ago

I'll look at it

On April 14, 2015 7:32:03 PM EDT, Andrew Nelson notifications@github.com wrote:

FrequencyByMonth should return an array of 12 elements, one frequency for each month. However, it seems the function is returning an array where each element is that month plus the previous month. That is, I'd expect to see something like [0, 0, 3, 2, 0, 5, 0, 1, 0, 0, 1, 2], but instead it seems to return something like [0, 0, 3, 5, 5, 10, 10, 11, 11, 11, 12, 14](compare the pattern to the first example array).


Reply to this email directly or view it on GitHub: https://github.com/jhinkel/SenseMaking/issues/1