mhlr / information-dynamics-toolkit

Automatically exported from code.google.com/p/information-dynamics-toolkit
GNU General Public License v3.0
0 stars 0 forks source link

KSG calculators to add noise in by default #42

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
I had left noise addition out by default so that results were repeatable. But 
with a small value of noise this should still be ok. Adding noise will stop us 
from getting crazy errors when systems converge to fixed attractors and we get 
lots of the same data points coming in. This has happened with Rose I think in 
2014, and on email to Sussex in 2015.
Consider this to make sure I'm not missing any good reason not to do it; then 
do it.

Original issue reported on code.google.com by joseph.lizier on 14 Apr 2015 at 4:10

GoogleCodeExporter commented 9 years ago
It also has another effect of skewing any Kd-trees built from such data, 
leading to stack overflow (which I found from bug report by Pedro in June 
2015). So I'm definitely making this happen by default now.

Original comment by joseph.lizier on 16 Jun 2015 at 11:53

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r653.

Original comment by joseph.lizier on 16 Jun 2015 at 12:01

GoogleCodeExporter commented 9 years ago
Verified by revision r655 (unit test updates)

Original comment by joseph.lizier on 16 Jun 2015 at 12:33