Skip to content

Commit ad553da

Browse files
SebastianAmentfacebook-github-bot
authored andcommitted
Deprecating GPyTorch's NanError, NotPSDError, psd_safe_cholesky in favor of LinearOperators's versions (#1377)
Summary: Pull Request resolved: #1377 Due to the recent migration to LinearOperators, we get a number of deprecation warnings, suggesting to switch from gpytorch versions of certain functions and objects to their LinearOperator equivalents. This diff aims to move all occurences of `NanError`, `NotPSDError`, `psd_safe_cholesky` in BoTorch to the new versions, thereby minimizing the number of deprecation warnings. Reviewed By: Balandat Differential Revision: D39287750 fbshipit-source-id: 5bf4afb603cc8165cc95027251b84290f4efd3a0
1 parent 2ecdbc1 commit ad553da

File tree

11 files changed

+15
-14
lines changed

11 files changed

+15
-14
lines changed

botorch/acquisition/cached_cholesky.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
from gpytorch.distributions.multitask_multivariate_normal import (
3030
MultitaskMultivariateNormal,
3131
)
32-
from gpytorch.utils.errors import NanError, NotPSDError
32+
from linear_operator.utils.errors import NanError, NotPSDError
3333
from torch import Tensor
3434

3535

botorch/acquisition/max_value_entropy_search.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,9 @@
4646
from botorch.models.utils import check_no_nans
4747
from botorch.sampling.samplers import SobolQMCNormalSampler
4848
from botorch.utils.transforms import match_batch_shape, t_batch_mode_transform
49-
from gpytorch.functions import inv_quad
50-
from gpytorch.utils.cholesky import psd_safe_cholesky
49+
50+
from linear_operator.functions import inv_quad
51+
from linear_operator.utils.cholesky import psd_safe_cholesky
5152
from scipy.optimize import brentq
5253
from scipy.stats import norm
5354
from torch import Tensor

botorch/fit.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
from botorch.settings import debug
2626
from gpytorch.mlls.marginal_log_likelihood import MarginalLogLikelihood
2727
from gpytorch.mlls.sum_marginal_log_likelihood import SumMarginalLogLikelihood
28-
from gpytorch.utils.errors import NotPSDError
28+
from linear_operator.utils.errors import NotPSDError
2929
from pyro.infer.mcmc import MCMC, NUTS
3030

3131

botorch/models/pairwise_gp.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,8 @@
4747
from gpytorch.module import Module
4848
from gpytorch.priors.smoothed_box_prior import SmoothedBoxPrior
4949
from gpytorch.priors.torch_priors import GammaPrior
50-
from gpytorch.utils.cholesky import psd_safe_cholesky
5150
from linear_operator.operators import LinearOperator
51+
from linear_operator.utils.cholesky import psd_safe_cholesky
5252
from scipy import optimize
5353
from torch import float32, float64, Tensor
5454
from torch.nn.modules.module import _IncompatibleKeys

botorch/optim/utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
from gpytorch.mlls.exact_marginal_log_likelihood import ExactMarginalLogLikelihood
2626
from gpytorch.mlls.marginal_log_likelihood import MarginalLogLikelihood
2727
from gpytorch.mlls.sum_marginal_log_likelihood import SumMarginalLogLikelihood
28-
from gpytorch.utils.errors import NanError, NotPSDError
28+
from linear_operator.utils.errors import NanError, NotPSDError
2929
from torch import Tensor
3030

3131

botorch/utils/gp_sampling.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
from botorch.models.multitask import MultiTaskGP
1919
from botorch.utils.sampling import manual_seed
2020
from gpytorch.kernels import Kernel, MaternKernel, RBFKernel, ScaleKernel
21-
from gpytorch.utils.cholesky import psd_safe_cholesky
21+
from linear_operator.utils.cholesky import psd_safe_cholesky
2222
from torch import Tensor
2323
from torch.distributions import MultivariateNormal
2424
from torch.nn import Module

botorch/utils/low_rank.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@
1313
from gpytorch.distributions.multitask_multivariate_normal import (
1414
MultitaskMultivariateNormal,
1515
)
16-
17-
from gpytorch.utils.cholesky import psd_safe_cholesky
18-
from gpytorch.utils.errors import NanError, NotPSDError
1916
from linear_operator.operators import BlockDiagLinearOperator, LinearOperator
17+
18+
from linear_operator.utils.cholesky import psd_safe_cholesky
19+
from linear_operator.utils.errors import NanError, NotPSDError
2020
from torch import Tensor
2121

2222

test/acquisition/test_cached_cholesky.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
from botorch.sampling.samplers import IIDNormalSampler
2121
from botorch.utils.low_rank import extract_batch_covar
2222
from botorch.utils.testing import BotorchTestCase, MockModel, MockPosterior
23-
from gpytorch.utils.errors import NanError, NotPSDError
23+
from linear_operator.utils.errors import NanError, NotPSDError
2424

2525
CHOLESKY_PATH = "linear_operator.operators._linear_operator.psd_safe_cholesky"
2626
EXTRACT_BATCH_COVAR_PATH = "botorch.acquisition.cached_cholesky.extract_batch_covar"

test/optim/test_utils.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@
4949
from gpytorch.mlls.sum_marginal_log_likelihood import SumMarginalLogLikelihood
5050
from gpytorch.priors.prior import Prior
5151
from gpytorch.priors.torch_priors import GammaPrior
52-
from gpytorch.utils.errors import NanError, NotPSDError
52+
from linear_operator.utils.errors import NanError, NotPSDError
5353

5454

5555
class DummyPrior(Prior):

test/test_fit.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@
2828
from gpytorch.likelihoods import GaussianLikelihood
2929
from gpytorch.mlls.exact_marginal_log_likelihood import ExactMarginalLogLikelihood
3030
from gpytorch.mlls.sum_marginal_log_likelihood import SumMarginalLogLikelihood
31-
from gpytorch.utils.errors import NanError, NotPSDError
3231
from gpytorch.utils.warnings import NumericalWarning
32+
from linear_operator.utils.errors import NanError, NotPSDError
3333
from scipy.optimize import OptimizeResult
3434

3535

0 commit comments

Comments
 (0)