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

Improving test coverage of UnifiedSkewNormal code #1408

Closed
wants to merge 2 commits into from

Conversation

SebastianAment
Copy link
Contributor

Summary: This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, see here.

Differential Revision: D39556258

@facebook-github-bot facebook-github-bot added CLA Signed Do not delete this pull request or issue due to inactivity. fb-exported labels Sep 21, 2022
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Sep 21, 2022

Codecov Report

Merging #1408 (6ef20f1) into main (98503e4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##             main    #1408    +/-   ##
========================================
  Coverage   99.98%   99.98%            
========================================
  Files         124      134    +10     
  Lines       11554    12382   +828     
========================================
+ Hits        11552    12380   +828     
  Misses          2        2            
Impacted Files Coverage Δ
botorch/utils/constants.py 100.00% <100.00%> (ø)
botorch/utils/probability/__init__.py 100.00% <100.00%> (ø)
botorch/utils/probability/bvn.py 100.00% <100.00%> (ø)
botorch/utils/probability/lin_ess.py 100.00% <100.00%> (ø)
botorch/utils/probability/linalg.py 100.00% <100.00%> (ø)
botorch/utils/probability/mvnxpb.py 100.00% <100.00%> (ø)
...utils/probability/truncated_multivariate_normal.py 100.00% <100.00%> (ø)
botorch/utils/probability/unified_skew_normal.py 100.00% <100.00%> (ø)
botorch/utils/probability/utils.py 100.00% <100.00%> (ø)
botorch/utils/safe_math.py 100.00% <100.00%> (ø)

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 23, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 7268c7f623a16ae2b2fe33d6a55a12d2aee9ca7e
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 26, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

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

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 26, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 4f3b956db52641b34d96642287c42982bdfdfe5f
SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 26, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 33e4371e37152c588132dd512c99f6b857d9caf2
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 26, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 83206e5a142c92841a036695acd859e51f7e74be
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 27, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 27, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 470cf3f5fc4f8004f7b65b33fd9e570ec927fc74
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 27, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 8441b859fcb67c5449e80f5815397dfda598feb7
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Sep 27, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 6, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: e62016ce08764fc669d158da6a64dc6bfcdc5fc2
SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 6, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 4e012054abf4ece1f1739dfc13eb051f09be828d
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 17, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

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

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 17, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 197c070f8b10abfc9a6bf7cb7873352b2edf48bd
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 17, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Differential Revision: D39556258

fbshipit-source-id: 9c27f49db2503ad0451cbdd7b90937349e334a9d
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 18, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Reviewed By: j-wilson

Differential Revision: D39556258

fbshipit-source-id: 96b62bf97254e7f54ba1089350f679cbf4ab4bda
SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 19, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Reviewed By: j-wilson

Differential Revision: D39556258

fbshipit-source-id: 0dbe542ea2013d56507096f18216fde82889d574
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

SebastianAment added a commit to SebastianAment/botorch that referenced this pull request Oct 19, 2022
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Reviewed By: j-wilson

Differential Revision: D39556258

fbshipit-source-id: a64f18cf56efbe706ba45ff0293b6aec9375ad07
jtwilson and others added 2 commits October 19, 2022 14:10
…rch#1394)

Summary:
Pull Request resolved: pytorch#1394

Introduces `utils/probability` submodule with the following offers:

- `bvn`: Methods for computing bivariate normal probabilities and moments.
- `MVNXPB`: Approximate solver for Multivariate Normal CDF.
- `LinearEllipticalSliceSampler`: Class for sampling trMVN random variables.
- `TruncatedMultivariateNormal`: Truncated multivariate normal Distribution class
- `UnifiedSkewNormal`: Unified skew normal Distribution class

Reviewed By: Balandat

Differential Revision: D39326106

fbshipit-source-id: d4f3346355ee9fc6312e04dfe17f0ae0427dbf2c
Summary:
Pull Request resolved: pytorch#1408

This commit improves the test coverage of the code located in botorch/utils/probability. For the current coverage without this commit, [see here](https://app.codecov.io/gh/pytorch/botorch/pull/1394).

Reviewed By: j-wilson

Differential Revision: D39556258

fbshipit-source-id: 07780fbb0902e82ca8cf06b3edda629f0ea9d1dd
@facebook-github-bot
Copy link
Contributor

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

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants