-
Notifications
You must be signed in to change notification settings - Fork 383
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Reap base_samples argument of GPyTorchPosterior.rsample #2254
Conversation
This pull request was exported from Phabricator. Differential Revision: D55038429 |
Summary: The `base_samples` argument to GPyTorchPosterior.rsample was deprecated in BoTorch 8.0.0 - reaped the deprecated code - removed the corresponding unit tests - Removed `base_samples` argument from `MockPosterior.rsample` used for testing, since the base Posterior class also does not permit `base_samples` in `rsample`. Moved an exception in `MockPosterior` so that invalid `base_samples` will still be checked. - fixed a docstring in PosteriorList Differential Revision: D55038429
f99ed3f
to
a66dfa7
Compare
This pull request was exported from Phabricator. Differential Revision: D55038429 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2254 +/- ##
==========================================
- Coverage 99.97% 99.97% -0.01%
==========================================
Files 197 197
Lines 17163 17154 -9
==========================================
- Hits 17159 17150 -9
Misses 4 4 ☔ View full report in Codecov by Sentry. |
Summary: X-link: facebookexternal/botorch_fb#17 The `base_samples` argument to GPyTorchPosterior.rsample was deprecated in BoTorch 8.0.0 - reaped the deprecated code - removed the corresponding unit tests - Removed `base_samples` argument from `MockPosterior.rsample` used for testing, since the base Posterior class also does not permit `base_samples` in `rsample`. Moved an exception in `MockPosterior` so that invalid `base_samples` will still be checked. - fixed a docstring in PosteriorList Reviewed By: saitcakmak Differential Revision: D55038429
Summary: X-link: facebookexternal/botorch_fb#17 The `base_samples` argument to GPyTorchPosterior.rsample was deprecated in BoTorch 8.0.0 - reaped the deprecated code - removed the corresponding unit tests - Removed `base_samples` argument from `MockPosterior.rsample` used for testing, since the base Posterior class also does not permit `base_samples` in `rsample`. Moved an exception in `MockPosterior` so that invalid `base_samples` will still be checked. - fixed a docstring in PosteriorList Reviewed By: saitcakmak Differential Revision: D55038429
This pull request has been merged in 4545f98. |
Summary:
The
base_samples
argument to GPyTorchPosterior.rsample was deprecated in BoTorch 8.0.0base_samples
argument fromMockPosterior.rsample
used for testing, since the base Posterior class also does not permitbase_samples
inrsample
. Moved an exception inMockPosterior
so that invalidbase_samples
will still be checked.Differential Revision: D55038429