Skip to content

Conversation

ethche
Copy link

@ethche ethche commented Aug 15, 2025

Summary: As pointed out in issue #2446 the example for model.condition_on_observations is outdated and raises an error when run. The docstring is now updated with proper usage of the method.

Differential Revision: D80371177

@meta-cla meta-cla bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Aug 15, 2025
@facebook-github-bot
Copy link
Contributor

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

Copy link

codecov bot commented Aug 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (b1097c6) to head (81e7926).

Additional details and impacted files
@@            Coverage Diff            @@
##              main     #2978   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          216       216           
  Lines        20310     20310           
=========================================
  Hits         20310     20310           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

ethche pushed a commit to ethche/botorch that referenced this pull request Aug 15, 2025
…orch#2978)

Summary:

As pointed out in [issue pytorch#2446](pytorch#2446) the example for model.condition_on_observations is outdated and raises an error when run. The docstring is now updated with proper usage of the method.

Reviewed By: Balandat

Differential Revision: D80371177
…orch#2978)

Summary:
Pull Request resolved: pytorch#2978

As pointed out in [issue pytorch#2446](pytorch#2446) the example for model.condition_on_observations is outdated and raises an error when run. The docstring is now updated with proper usage of the method.

Reviewed By: Balandat

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

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

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