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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Register torch.linalg.LinAlgError to pyro exception handling #1607

Closed
wants to merge 2 commits into from

Conversation

Balandat
Copy link
Contributor

@Balandat Balandat commented Jan 4, 2023

Summary: Uses draft changes from pyro-ppl/pyro#3168 (part of pyro 1.8.4 pulled in via D42331876) to register handling of torch.linalg.LinAlgError and the ValueError that can be raised in the torch distribution's __init__()

Differential Revision: D42159791

Summary:
This includes two major changes of interest to botorch:
1. Support for pytorch 2.0 (allowing us to not require pyro dev branch when testing against pytorch nightlies): pyro-ppl/pyro#3164
2. The ability to registering of custom exception handlers for numerical issues: pyro-ppl/pyro#3168

Pull Request resolved: pytorch#1606

Reviewed By: saitcakmak

Differential Revision: D42330914

Pulled By: Balandat

fbshipit-source-id: 440d9ff00df60f4ba88acc82e323f9e675caf254
Summary: Uses draft changes from pyro-ppl/pyro#3168 (part of pyro 1.8.4 pulled in via D42331876) to register handling of `torch.linalg.LinAlgError` and the `ValueError` that can be raised in the torch distribution's `__init__()`

Differential Revision: D42159791

fbshipit-source-id: ef1f53c9471ba8b1d62ca5ad347e5a471d9bb7a1
@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels Jan 4, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D42159791

@codecov
Copy link

codecov bot commented Jan 4, 2023

Codecov Report

Merging #1607 (35df664) into main (1addd3a) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main     #1607   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          154       154           
  Lines        13775     13768    -7     
=========================================
- Hits         13775     13768    -7     
Impacted Files Coverage Δ
botorch/models/fully_bayesian.py 100.00% <100.00%> (ø)
botorch/models/fully_bayesian_multitask.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in cbbbf11.

@Balandat Balandat deleted the export-D42159791 branch March 5, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants