Skip to content

Commit f7a210c

Browse files
committed
Merge pull request #1 from joeyism/master
Added longer eval periods
2 parents 37a16df + efcbc55 commit f7a210c

3 files changed

Lines changed: 8 additions & 151 deletions

File tree

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,4 +38,4 @@ console.log(binner.hist);
3838

3939
Possible bin sizes are: **day, hour, minute, second**
4040

41-
Available evaluation periods are: **week, day, hour**
41+
Available evaluation periods are: **year**, **month**, **week, day, hour**

npm-debug.log

Lines changed: 0 additions & 149 deletions
This file was deleted.

timestamp-binning.js

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,12 @@ module.exports = function(evalPeriodString, binSizeString, excludeBursts) {
4545
//evalPeriod also corresponds to the number of possible bins
4646
//week, day, hour
4747
switch (evalPeriodString.toLowerCase()) {
48+
case "year":
49+
evalPeriod = 31536000000 / binSize;
50+
break;
51+
case "month":
52+
evalPeriod = 2592000000 / binSize;
53+
break;
4854
case "week":
4955
evalPeriod = 604800000 / binSize;
5056
break;
@@ -117,4 +123,4 @@ module.exports = function(evalPeriodString, binSizeString, excludeBursts) {
117123
}
118124
}
119125
};
120-
};
126+
};

0 commit comments

Comments
 (0)