twitter / AnomalyDetection

Anomaly Detection with R
GNU General Public License v3.0
3.55k stars 776 forks source link

Remove use of eval in favor of first class function #44

Open zrnsm opened 9 years ago

zrnsm commented 9 years ago

There seems to be an unnecessary use of eval for timestamp aggregation. This patch uses the sum function directly. The other changes are just removal of unnecessary trailing whitespace.

akejariwal commented 9 years ago

@caniszczyk Can you please merge this. I approve the merge.

CLAassistant commented 5 years ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.