mattpoggi / mono-uncertainty

CVPR 2020 - On the uncertainty of self-supervised monocular depth estimation

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Problem with Bayesian uncertainty method

brandonwagstaff opened this issue · comments

Hi There,

I'm wondering if there is a discrepancy between this line of code and equation 17 in the paper that computes the full uncertainty for the Bayesian approach:

pred_uncert = torch.var(disps_distribution, dim=0, keepdim=False) + torch.sum(torch.cat(uncerts_distribution, 0), dim=0, keepdim=False)

As far as I'm aware, the empirical variance from the ensemble should be added with the square of the learned uncertainty output. Am I correct in associating this line of code with equation 17 of the paper? If so, there might be an issue here. I was wondering if this affects the visualization of uncertainties in the paper.

Thanks!

Hi @brandonwagstaff,
yep, that's related to #11 , a missing torch.exp (and squared term as well) was missing in the first code release.
I'm fixing this, thanks for pointing it out.

As you correctly point out, this affects the colormap scaling of the qualitative results, while numerically does not make a real difference (usually, the second decimal in RMSE metrics).

Got it, thanks for the response!