Boost C++ Libraries: Ticket #9712: Documentation errors

owner, component changed

Steven Watanabe — Mon, 10 Mar 2014 22:44:36 GMT

owner changed from Matias Capeletto to John Maddock
component Documentation → math

I think math is the right component, but a link to the page containing the problem would be helpful.

owner changed

John Maddock — Tue, 11 Mar 2014 17:05:38 GMT

owner changed from John Maddock to Paul A. Bristow

Paul, can I get you to check these out? Thanks!

Paul A. Bristow — Sun, 16 Mar 2014 17:55:48 GMT

Sorry for delay in response. Thanks for reporting this mistake, and not really just in the documentation :-(

Do you have a view of which constant is useful. I can't find where the constant called pow23_four_minus_pi is used.

It clearly should be the correct name/value combination.

Is either or both useful - especially to you?

pow23_four_minus_pi

pow32_four_minus_pi

(or both)

John Maddock — Sun, 16 Mar 2014 18:39:10 GMT

A quick grep shows it was used in the Rayleigh distribution at one point, but is commented out. Maybe we should just remove it - I doubt it has much general use?

John Maddock — Sun, 16 Mar 2014 18:52:57 GMT

On second thoughts, maybe

template <class RealType, class Policy>
inline RealType kurtosis(const rayleigh_distribution<RealType, Policy>& /*dist*/)

Should really use the constant? albeit we need to check both the name and the value to see which is correct ;)

Paul A. Bristow — Mon, 17 Mar 2014 17:59:44 GMT

The constant pow23_four_minus_pi does not seem generally useful and is not necessary for Rayleigh skewness or kurtosis.

So the constant has been removed and an explicit calculation used for the kurtosis excess(and skewness). This agrees with the constant calculated using NTL previously.

status, milestone changed; resolution set

Paul A. Bristow — Mon, 17 Mar 2014 18:00:43 GMT

status new → closed
resolution → fixed
milestone To Be Determined → Boost 1.56.0