baudren / montepython_public

Public repository for the Monte Python Code
MIT License
65 stars 114 forks source link

bug in applying non-flat prior #119

Open seshnadathur opened 5 years ago

seshnadathur commented 5 years ago

The code gives an error when the user attempts to use a Gaussian prior for any nuisance parameter, by including a line like data.parameters['xyz'] = [10.0, 0, 20, 0.5, 1, 'nuisance', 'gaussian', 10.0, 1] in the input parameter file (as the Prior module is designed to read, although this is not explained very well in the documentation).

If this line is included, the parameter xyz is then not recognised as a nuisance parameter. The cause of this bug is in line 999 of the file data.py, which should read: self['role'] = array[5] In the current implementation it is taking the provided sigma value of the prior as the parameter role.

brinckmann commented 5 years ago

Thanks! I'll fix it in the upcoming release. By the way, this github page is no longer being maintained, so please report bugs on our new github page: https://github.com/brinckmann/montepython_public Best, Thejs