Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Bug fix: _filter_kwargs was erroring when provided a function without a __name__ attribute #1678

Closed
wants to merge 1 commit into from

Conversation

esantorella
Copy link
Member

Summary: See #1667

Differential Revision: D43286116

…ut a `__name__` attribute

Summary: See pytorch#1667

Differential Revision: D43286116

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

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

@codecov
Copy link

codecov bot commented Feb 14, 2023

Codecov Report

Merging #1678 (4e6a119) into main (ad38736) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 4e6a119 differs from pull request most recent head f9b02c5. Consider uploading reports for the commit f9b02c5 to get more accurate results

@@            Coverage Diff            @@
##              main     #1678   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          169       169           
  Lines        14517     14518    +1     
=========================================
+ Hits         14517     14518    +1     
Impacted Files Coverage Δ
botorch/optim/utils/common.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 63dd0cd.

# for free to join this conversation on GitHub. Already have an account? # 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