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

qLogNEI **kwargs removal #2406

Closed
wants to merge 3 commits into from
Closed

Conversation

hvarfner
Copy link
Contributor

@hvarfner hvarfner commented Jul 1, 2024

Summary: Removed the ability for qLogNEI to swallow unused kwargs.

Differential Revision: D59238948

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

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

Copy link

codecov bot commented Jul 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.98%. Comparing base (7173b39) to head (82d6e2d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2406   +/-   ##
=======================================
  Coverage   99.98%   99.98%           
=======================================
  Files         189      189           
  Lines       16666    16666           
=======================================
  Hits        16664    16664           
  Misses          2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@facebook-github-bot
Copy link
Contributor

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 2, 2024
Summary:
Pull Request resolved: pytorch#2406

Removed the ability for qLogNEI to swallow unused kwargs.

Reviewed By: saitcakmak

Differential Revision: D59238948
@hvarfner hvarfner force-pushed the export-D59238948 branch from b0fd166 to a577eda Compare July 2, 2024 15:18
@facebook-github-bot
Copy link
Contributor

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 2, 2024
Summary:
Pull Request resolved: pytorch#2406

Removed the ability for qLogNEI to swallow unused kwargs.

Reviewed By: saitcakmak

Differential Revision: D59238948
@hvarfner hvarfner force-pushed the export-D59238948 branch from a577eda to 72e2596 Compare July 2, 2024 15:29
@facebook-github-bot
Copy link
Contributor

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

hvarfner pushed a commit to hvarfner/botorch that referenced this pull request Jul 2, 2024
Summary:
Pull Request resolved: pytorch#2406

Removed the ability for qLogNEI to swallow unused kwargs.

Reviewed By: saitcakmak

Differential Revision: D59238948
@hvarfner hvarfner force-pushed the export-D59238948 branch from 72e2596 to 942279a Compare July 2, 2024 15:33
Summary:
Pull Request resolved: pytorch#2406

Removed the ability for qLogNEI to swallow unused kwargs.

Reviewed By: saitcakmak

Differential Revision: D59238948
@facebook-github-bot
Copy link
Contributor

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

@hvarfner hvarfner force-pushed the export-D59238948 branch from 942279a to 640d83e Compare July 2, 2024 15:38
@saitcakmak
Copy link
Contributor

The docs failure is unrelated and should be fixed by a diff that just landed.

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 0455dc3.

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.

3 participants