Skip to content

Add outcome and feature names to datasets #2015

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

Closed
wants to merge 1 commit into from

Conversation

saitcakmak
Copy link
Contributor

Summary:
This will help us while passing data around in Ax by offering more granular information on what each dataset represents.

Also hard-deprecates dict_from_iter method that was unused outside of tests.

Differential Revision: D48926319

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

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

saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Sep 18, 2023
Summary:

This will help us while passing data around in Ax by offering more granular information on what each dataset represents.

Also hard-deprecates `dict_from_iter` method that was unused outside of tests.

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

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

@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #2015 (25ef453) into main (ceb686d) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

❗ Current head 25ef453 differs from pull request most recent head 39940ad. Consider uploading reports for the commit 39940ad to get more accurate results

@@            Coverage Diff            @@
##              main     #2015   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          179       179           
  Lines        15807     15803    -4     
=========================================
- Hits         15807     15803    -4     
Files Changed Coverage Δ
botorch/utils/datasets.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

saitcakmak added a commit to saitcakmak/Ax that referenced this pull request Sep 18, 2023
Summary: Follows up pytorch/botorch#2015 to propagate the changes in Ax and fixes the tests.

Differential Revision: D49340640

fbshipit-source-id: bce2bed5639b8f816c74fbc32ee3145d43b33a9e
saitcakmak added a commit to saitcakmak/botorch that referenced this pull request Sep 18, 2023
Summary:

This will help us while passing data around in Ax by offering more granular information on what each dataset represents.

Also hard-deprecates `dict_from_iter` method that was unused outside of tests.

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

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

Summary:

This will help us while passing data around in Ax by offering more granular information on what each dataset represents.

Also hard-deprecates `dict_from_iter` method that was unused outside of tests.

Reviewed By: Balandat

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

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

saitcakmak added a commit to saitcakmak/Ax that referenced this pull request Sep 19, 2023
…k#1853)

Summary:
Pull Request resolved: facebook#1853

Follows up pytorch/botorch#2015 to propagate the changes in Ax and fixes the tests.

Reviewed By: Balandat

Differential Revision: D49340640

fbshipit-source-id: 81a51ce81d4d1977e88365f73065f9214ca3d48b
facebook-github-bot pushed a commit to facebook/Ax that referenced this pull request Sep 19, 2023
Summary:
Pull Request resolved: #1853

Follows up pytorch/botorch#2015 to propagate the changes in Ax and fixes the tests.

Reviewed By: Balandat

Differential Revision: D49340640

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

This pull request has been merged in aa13c14.

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