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

Use psd_safe_cholesky in MES #518

Closed
wants to merge 1 commit into from

Conversation

Balandat
Copy link
Contributor

Summary: Avoid singularity errors with torch.cholesky

Reviewed By: dme65, qingfeng10

Differential Revision: D23171308

Summary: Avoid singularity errors with torch.cholesky

Reviewed By: dme65, qingfeng10

Differential Revision: D23171308

fbshipit-source-id: 27db264cb44c5c3b1b8dab23728c642540290235
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Aug 17, 2020

Codecov Report

Merging #518 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #518   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           84        84           
  Lines         5274      5275    +1     
=========================================
+ Hits          5274      5275    +1     
Impacted Files Coverage Δ
botorch/acquisition/max_value_entropy_search.py 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update da84299...10e878b. Read the comment docs.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 781ea14.

facebook-github-bot pushed a commit that referenced this pull request Sep 11, 2020
Summary:
#### New Features
* Constrained Multi-Objective tutorial (#493)
* Multi-fidelity Knowledge Gradient tutorial (#509)
* Support for batch qMC sampling (#510)
* New `evaluate` method for `qKnowledgeGradient` (#515)

#### Compatibility
* Require PyTorch >=1.6 (#535)
* Require GPyTorch >=1.2 (#535)
* Remove deprecated `botorch.gen module` (#532)

#### Bug fixes
* Fix bad backward-indexing of task_feature in `MultiTaskGP` (#485)
* Fix bounds in constrained Branin-Currin test function (#491)
* Fix max_hv for C2DTLZ2 and make Hypervolume always return a float (#494)
* Fix bug in `draw_sobol_samples` that did not use the proper effective dimension (#505)
* Fix constraints for `q>1` in `qExpectedHypervolumeImprovement` (c80c4fd)
* Only use feasible observations in partitioning for `qExpectedHypervolumeImprovement`
  in `get_acquisition_function` (#523)
* Improved GPU compatibility for `PairwiseGP` (#537)

#### Performance Improvements
* Reduce memory footprint in `qExpectedHypervolumeImprovement` (#522)
* Add `(q)ExpectedHypervolumeImprovement` to nonnegative functions
  [for better initialization] (#496)

#### Other changes
* Support batched `best_f` in `qExpectedImprovement` (#487)
* Allow to return full tree of solutions in `OneShotAcquisitionFunction` (#488)
* Added `construct_inputs` class method to models to programmatically construct the
  inputs to the constructor from a standardized `TrainingData` representation
  (#477, #482, 3621198)
* Acqusiition function constructors now accept catch-all `**kwargs` options
  (#478, e5b6935)
* Use `psd_safe_cholesky` in `qMaxValueEntropy` for better numerical stabilty (#518)
* Added `WeightedMCMultiOutputObjective` (81d91fd)
* Add ability to specify `outcomes` to all multi-output objectives (#524)
* Return optimization output in `info_dict` for `fit_gpytorch_scipy` (#534)
* Use `setuptools_scm` for versioning (#539)

Pull Request resolved: #542

Reviewed By: sdaulton

Differential Revision: D23645619

Pulled By: Balandat

fbshipit-source-id: 0384f266cbd517aacd5778a6e2680336869bb31c
@Balandat Balandat deleted the export-D23171308 branch December 2, 2020 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants