search

rabauke / trng4

state of the art C++ pseudo-random number generator library for sequential and parallel Monte Carlo simulations
https://www.numbercrunch.de/trng/
BSD 3-Clause "New" or "Revised" License
120 stars 21 forks source link

Weibull constructor documentation mistake #17

Closed vchz closed 4 years ago

vchz commented 4 years ago

Hello, Thanks a lot for your library, it has been very useful to me so far and I really don't know how I would proceed without it. I use version 4.22. I've noticed an inconsistency between the constructor in weibull_dist.hpp and the documentation. Nothing big, but though a bit sill, maybe you are aware of it already.

In the 4.22 the constructor is declared as : weibull_dist(result_type theta, result_type beta) While in the documentation it is said to be : weibull_dist(result_type beta, result_type theta)

There is as far as I tested, no problem with this weibull distribution as long as you use the good declaration (the one of the hpp file).

Thanks ! Victor

rabauke commented 4 years ago

Thanks for drawing my attention to this documentation issue. Has been fixed in the development branch.