search

katylettuce / beast-mcmc

Automatically exported from code.google.com/p/beast-mcmc
0 stars 0 forks source link

RandomWalkOperator is reflecting negative value to positive #683

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
This happens when the lower parameter boundary is set by default (to 
-Double.MAX_VALUE).  Note that this limit is also a bug since -Double.MAX_VALUE 
!= Double.MIN_VALUE

Original issue reported on code.google.com by msuch...@gmail.com on 26 Mar 2013 at 3:24

GoogleCodeExporter commented 9 years ago

Original comment by msuch...@gmail.com on 26 Mar 2013 at 3:29

GoogleCodeExporter commented 9 years ago 
Double.MIN_VALUE is the smallest _positive_ number. I guess the use of 
-Double.MAX_VALUE was to give the largest negative number (although it is 
possible that this is not the correct way of denoting this). I think the 
default bounds should be NEGATIVE_INFINITY to POSITIVE_INFINITY.

Original comment by ramb...@gmail.com on 23 Jan 2014 at 12:25

GoogleCodeExporter commented 9 years ago 
Reopening this issue. Do we have an example of incorrect behaviour?

Original comment by ramb...@gmail.com on 23 Jan 2014 at 1:00

GoogleCodeExporter commented 9 years ago 
Probably fixed in revision 6132 but not tested because of lack of test case.

Original comment by ramb...@gmail.com on 23 Jan 2014 at 4:49

GoogleCodeExporter commented 9 years ago 
Marking as fixed. Reopen if there is a test case.

Original comment by ramb...@gmail.com on 8 Aug 2014 at 1:06