check if probability is in [0, 1] - Githubissues

boostorg / accumulators

An awesome library from Boost

http://boost.org/libs/accumulators

22 stars 54 forks source link

check if probability is in [0, 1] #10

Open e-kwsm opened 7 years ago

e-kwsm commented 7 years ago

include/boost/accumulators/statistics/extended_p_square.hpp
include/boost/accumulators/statistics/extended_p_square_quantile.hpp
include/boost/accumulators/statistics/extended_p_square_quantile.hpp
include/boost/accumulators/statistics/p_square_quantile.hpp
include/boost/accumulators/statistics/peaks_over_threshold.hpp
include/boost/accumulators/statistics/pot_quantile.hpp
include/boost/accumulators/statistics/pot_tail_mean.hpp
include/boost/accumulators/statistics/tail_mean.hpp
include/boost/accumulators/statistics/tail_quantile.hpp
include/boost/accumulators/statistics/tail_variate_means.hpp
include/boost/accumulators/statistics/weighted_extended_p_square.hpp
include/boost/accumulators/statistics/weighted_peaks_over_threshold.hpp
include/boost/accumulators/statistics/weighted_tail_mean.hpp
include/boost/accumulators/statistics/weighted_tail_quantile.hpp
include/boost/accumulators/statistics/weighted_tail_variate_means.hpp

yuvalif commented 5 years ago

@e-kwsm don't think you can introduce exceptions into existing code like that. also, there is the runtime check cost. Instead, would recommend adding documentation on the behavior in case of invalid probability values.