search

sbi-benchmark / sbibm

Simulation-based inference benchmark
https://sbi-benchmark.github.io
MIT License
88 stars 34 forks source link

Bugfix for likelihood of two_moons if log=False #41

Closed michaeldeistler closed 2 years ago

michaeldeistler commented 2 years ago

there was a bug in the likelihood of the two-moons simulator. It only affects the case where log=False. Without the suggested change, the exp() is computed twice: Once in the line in which I suggested the change and once here

jan-matthis commented 2 years ago

Great, thanks for catching and fixing this -- much appreciated!