botorch icon indicating copy to clipboard operation
botorch copied to clipboard

Removing custom `BlockDiagLazyTensor` logic when using `Standardize` (Take 2)

Open SebastianAment opened this issue 3 years ago • 2 comments

Summary: Due to this linear operator PR, we should now be able to remove the custom logic in Standardize without performance impact.

Differential Revision: D39746709

SebastianAment avatar Sep 22 '22 22:09 SebastianAment

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

facebook-github-bot avatar Sep 22 '22 22:09 facebook-github-bot

Codecov Report

Merging #1414 (d9fabef) into main (a6cc512) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main     #1414   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          134       134           
  Lines        12408     12403    -5     
=========================================
- Hits         12408     12403    -5     
Impacted Files Coverage Δ
botorch/models/transforms/outcome.py 100.00% <100.00%> (ø)

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

codecov[bot] avatar Sep 22 '22 22:09 codecov[bot]

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

facebook-github-bot avatar Nov 07 '22 22:11 facebook-github-bot

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

facebook-github-bot avatar Nov 08 '22 15:11 facebook-github-bot

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

facebook-github-bot avatar Nov 08 '22 17:11 facebook-github-bot