gonum / stat

73%

master: 74%

LAST BUILD ON BRANCH fixcovrace

branch: fixcovrace

CHANGE BRANCH

x

Reset

• fixcovrace
• add-go.1.7-drop-go-1.4
• add-numparameters
• addbernouilli
• addcat
• addchisquared
• addcombin
• addexpected
• addgamma
• addlognormal
• addmahal
• addmarg
• addmargsing
• addmvgauss
• addmvnorm
• addmvt
• addnormquant
• addquantiles
• addquants
• addrsquared
• addsamplemv
• addtransformgauss
• addwishart
• allocs
• archive
• batchsample
• binomial-overflow
• blas64
• build-status-only-master
• cc-example
• cheaper-ca
• cleanup-wishart
• close
• correlationmatrix
• covariance-matrix
• covariancematrix-multrans
• covdoc
• coveralls-badge
• covsym
• deprecate
• descale
• dirichlet
• docfix
• docs
• equal
• equalapprox
• errors
• expandstudents
• featuresample
• fix-marshaller-spelling
• fixbetacomment
• fixcol
• fixcovman
• fixdistuv
• fixgamma
• fixhisto
• fixlatin
• fixmax
• fixmul
• fixnormal
• fixnormchol
• fixuniform
• fixunifprob
• go-1.5.4-and-1.6.1
• go-1.6.3-and-1.7rc2
• go-1.7beta2
• go-1.7rc3
• go1.6beta2
• go1.6rc1
• go1.6rc2
• go1.7beta1
• incremental-stats
• internalize-parameters
• keepsigma
• master
• movequant
• mvinterface
• mvtmargsing
• names
• no-overload
• normalcondition
• normcondpan
• normprec
• only-coveralls-on-gonum-branch
• pca
• rc1
• regression
• remove-marshaler
• remove-redundant-types
• renamescore
• scaled-symouterk
• setsigma
• simplify-sigs
• statdist
• statdist4
• statdistunif
• sudo-false
• suffstat
• tcopy-fix
• test-coverage-improvement
• travis-gimme
• travis-go-1.4.2
• travis-go-1.6.2
• trim
• typo
• unifbound
• unifent
• use-multrans
• weighted

Build # 909

Build Type

push

travis-ci

Committed by btracey

Commit Message

Remove race in checking for sigma

Methods in Normal and StudentsT should either use sigma or not, they shouldn't conditionally use sigma depending on if it has been set. The current code races with setSigma(). Fixing it with a mutex is not strictly a data race, but it does mean the behavior of the program depends on execution order. Remove this behavior entirely by just computing the relevant entry of the covariance matrix.

Run Details

2449 of 3345 relevant lines covered (73.21%)

2.5 hits per line