Free cookie consent management tool by TermsFeed Policy Generator

Opened 9 years ago

Closed 9 years ago

#2502 closed defect (done)

Calculation of confidence bounds for Gaussian process models seems incorrect

Reported by: gkronber Owned by: gkronber
Priority: medium Milestone: HeuristicLab 3.3.13
Component: Algorithms.DataAnalysis Version: 3.3.12
Keywords: Cc:


The confidence bounds shown in the line chart do not correspond with sigma noise in the model (see attachment).

Attachments (1)

constant_gp_model.jpg (64.3 KB) - added by gkronber 9 years ago.
constant model (noise sigma = 0.65)

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by gkronber

constant model (noise sigma = 0.65)

comment:1 Changed 9 years ago by gkronber

r13121: calculate the variance for the noisy test data instead V(y*) instead of the variance for the posterior GP function V(f*)

comment:2 Changed 9 years ago by gkronber

  • Owner set to gkronber
  • Status changed from new to accepted

comment:3 Changed 9 years ago by gkronber

  • Owner changed from gkronber to mkommend
  • Status changed from accepted to reviewing

comment:4 Changed 9 years ago by mkommend

  • Owner changed from mkommend to gkronber
  • Status changed from reviewing to readytorelease

Reviewed r13121. OK.

The changes in the GP solution line chart are pretty clear. One thing that came to my mind while reviewing is that it should be possible to specify the confidence interval (default is set to 0.95).

The change in the GP-Model class is hard to judge without actually reading the paper (which I don't have at hand). I'll trust you that the noise variance has to be added to calculate the variance of the estimated values.

comment:5 Changed 9 years ago by gkronber

Rasmussen et al. "Gaussian Processes for Machine Learning" chapter 2, page 19: "The algorithm returns the predictive mean and variance for noise free test data - to compute the predictive distribution for noisy test data y∗, simply add the noise variance σ²n to the predictive variance of f∗." (

comment:6 Changed 9 years ago by gkronber

  • Resolution set to done
  • Status changed from readytorelease to closed

r13145: merged r13121 from trunk to stable

Note: See TracTickets for help on using tickets.