twitter / AnomalyDetection

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

Minor performance improvements. #17

Closed wrathematics closed 9 years ago

wrathematics commented 9 years ago

Added the bytecode compiler, and avoided some unneeded memory allocations and castings. These improvements are very minor, but they're basically free. You're taking a major performance hit from all the [ calls, but it would take a lot of work to avoid this overhead. I may think some more about this, assuming you're at all interested.

Also switched T and F to TRUE and FALSE respectively, as the former two are reassignable.

owenvallis commented 9 years ago

Thanks for these wrathematics. Any other improvements you would also certainly be welcome.

Cheers,

jhochenbaum commented 9 years ago

Thanks @wrathematics !