Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docs] #2221

Open
max-gains opened this issue Dec 8, 2022 · 1 comment
Open

[Docs] #2221

max-gains opened this issue Dec 8, 2022 · 1 comment

Comments

@max-gains
Copy link

馃摎 Documentation/Examples

I think the docs for deep multi-output regression are wrong:
https://docs.gpytorch.ai/en/stable/examples/05_Deep_Gaussian_Processes/DGP_Multitask_Regression.html

This example uses only a scaled RBF kernel (not a multi-output kernel) and a MultivariateNormal dist, not a MultitaskMultivariateNormal.
There are also differences between the code and the supporting writing (which says a MultitaskMultivariateNormal should be used).
I can provide a fix if requested.

@gpleiss
Copy link
Member

gpleiss commented Dec 9, 2022

This example uses only a scaled RBF kernel (not a multi-output kernel) and a MultivariateNormal dist, not a MultitaskMultivariateNormal.

This is actually correct. The DeepGP outputs multiple independent MultitaskNormal distributions (based on the width of the last layer) which are then combined into a MultitaskMultivariateNormal.

There are also differences between the code and the supporting writing (which says a MultitaskMultivariateNormal should be used).

I understand that the docs might be confusing, so if you have suggestions for how to make them clearer we'd be open to a fix!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants