Skip to content

Conversation

@saitcakmak
Copy link
Contributor

Summary:
This model has been buggy for quite a long time and we still haven't fixed it. Removing it should be preferable to keeping around a known buggy model. Example bug reports:

Differential Revision: D65543676

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

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

saitcakmak added a commit to saitcakmak/Ax that referenced this pull request Nov 6, 2024
Summary:
X-link: meta-pytorch/botorch#2616

This model has been buggy for quite a long time and we still haven't fixed it. Removing it should be preferable to keeping around a known buggy model. Example bug reports:
- meta-pytorch/botorch#861
- meta-pytorch/botorch#933
- meta-pytorch/botorch#2551

Differential Revision: D65543676
@codecov
Copy link

codecov bot commented Nov 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.98%. Comparing base (cd657d9) to head (232aae1).
Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2616      +/-   ##
==========================================
- Coverage   99.98%   99.98%   -0.01%     
==========================================
  Files         196      196              
  Lines       17368    17328      -40     
==========================================
- Hits        17366    17326      -40     
  Misses          2        2              

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

saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Nov 6, 2024
Summary:

This model has been buggy for quite a long time and we still haven't fixed it. Removing it should be preferable to keeping around a known buggy model. Example bug reports:
- meta-pytorch#861
- meta-pytorch#933
- meta-pytorch#2551

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

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

@saitcakmak saitcakmak changed the title Remove HeterogeneousSingleTaskGP Remove HeteroskedasticSingleTaskGP Nov 6, 2024
Summary:

This model has been buggy for quite a long time and we still haven't fixed it. Removing it should be preferable to keeping around a known buggy model. Example bug reports:
- meta-pytorch#861
- meta-pytorch#933
- meta-pytorch#2551

Reviewed By: esantorella

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 60387d0.

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