search

rehanvdm / serverless-website-analytics

A CDK construct that consists of a serverless backend, frontend and client side code to track website analytics
GNU General Public License v2.0
167 stars 13 forks source link

feat: Static anomaly detection for pr #89

Closed kc0bfv closed 4 months ago

kc0bfv commented 5 months ago

This does not directly address issue #75, but does address some comments there.

This PR adds a minimumViews parameter which must be exceeded before the anomaly detection considers a set of views to be an anomaly. The default is 0, so by default this is off. An example is provided in the anomaly detection documentation.

rehanvdm commented 5 months ago

I ran the 'Simulate and predict' test in tests/application/backend/cron-anomaly-detection/index.ts with:

Value at 0: image

Value at 30: image

It does a good job of filtering out those small spikes that should not be considered anomalies, thanks!

kc0bfv commented 4 months ago

Updated! Sorry that took so long - vacation, work, blah blah. Actually I didn't realize GitHub made it so easy to commit recommendations like that!

rehanvdm commented 4 months ago

No problem, I know how it goes and this is open source, so no obligation. Thanks again.

github-actions[bot] commented 4 months ago

:tada: This PR is included in version 1.11.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: