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

Remove file emptied in previous diff #2040

Closed
wants to merge 2 commits into from

Conversation

esantorella
Copy link
Member

Summary: Removes numpy_converter.py.

Differential Revision: D50203838

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 12, 2023
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Oct 12, 2023

Codecov Report

Merging #2040 (614c61e) into main (6d330eb) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 614c61e differs from pull request most recent head c416ba9. Consider uploading reports for the commit c416ba9 to get more accurate results

@@             Coverage Diff             @@
##              main    #2040      +/-   ##
===========================================
- Coverage   100.00%   99.98%   -0.02%     
===========================================
  Files          179      178       -1     
  Lines        15889    15755     -134     
===========================================
- Hits         15889    15752     -137     
- Misses           0        3       +3     
Files Coverage Δ
botorch/optim/__init__.py 100.00% <ø> (ø)
botorch/optim/fit.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 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 was exported from Phabricator. Differential Revision: D50203838

esantorella added a commit to esantorella/botorch that referenced this pull request Oct 12, 2023
Summary:
Pull Request resolved: pytorch#2040

Removes numpy_converter.py.

Differential Revision: D50203838

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

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

esantorella added a commit to esantorella/botorch that referenced this pull request Oct 12, 2023
Summary:
Pull Request resolved: pytorch#2040

Removes numpy_converter.py.

Differential Revision: D50203838

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

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

esantorella added a commit to esantorella/botorch that referenced this pull request Oct 12, 2023
Summary:
Pull Request resolved: pytorch#2040

Removes numpy_converter.py.

Reviewed By: saitcakmak

Differential Revision: D50203838

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

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

esantorella added a commit to esantorella/botorch that referenced this pull request Oct 13, 2023
Summary:
Pull Request resolved: pytorch#2040

Removes numpy_converter.py.

Reviewed By: saitcakmak

Differential Revision: D50203838

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

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

esantorella added a commit to esantorella/botorch that referenced this pull request Oct 13, 2023
Summary:
Pull Request resolved: pytorch#2040

Removes numpy_converter.py.

Reviewed By: saitcakmak

Differential Revision: D50203838

fbshipit-source-id: b9dbe4e7cccc0883df4b79e111690a04c3646abb
…rch (pytorch#1995)

Summary:
## Motivation

Removes most of what was deprecated in pytorch#1439, in version 0.8.0, except for `_get_extra_mll_args` and `fit_gpytorch_model`. Removal was straightforward since the functionality was not used and relevant unit tests were clearly labeled and self-contained.

### Have you read the [Contributing Guidelines on pull requests](https://github.com/pytorch/botorch/blob/main/CONTRIBUTING.md#pull-requests)?

Yes

Pull Request resolved: pytorch#1995

Pull Request resolved: pytorch#2041

Test Plan:
Existing units; make sure codecov has not regressed from deleting tests.

## Related PRs

pytorch#1439

Differential Revision: https://internalfb.com/D50276821

fbshipit-source-id: 2c90b1b254a5246160b9848812155de68b2a00ce
Summary:
Pull Request resolved: pytorch#2040

Removes numpy_converter.py.

Reviewed By: saitcakmak

Differential Revision: D50203838

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 8413920.

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