Skip to content

Use get_heterogeneous_feature_mapping in LFI argparse dispatcher (#5195)#5195

Open
hvarfner wants to merge 4 commits into
facebook:mainfrom
hvarfner:export-D102197138
Open

Use get_heterogeneous_feature_mapping in LFI argparse dispatcher (#5195)#5195
hvarfner wants to merge 4 commits into
facebook:mainfrom
hvarfner:export-D102197138

Conversation

@hvarfner
Copy link
Copy Markdown

@hvarfner hvarfner commented Apr 29, 2026

Summary:

Refactors the learned imputation argument dispatcher to delegate feature index computation to the dataset's built-in mapping utility. This eliminates duplicated feature-ordering logic and ensures consistency with the canonical ordering convention.

Differential Revision: D102197138

@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Apr 29, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Apr 29, 2026

@hvarfner has exported this pull request. If you are a Meta employee, you can view the originating Diff in D102197138.

Carl Hvarfner added 4 commits April 29, 2026 07:57
…TaskGP (facebook#5192)

Summary:
X-link: meta-pytorch/botorch#3296


Automatically configures learned feature imputation for models that pad heterogeneous per-task data to the full joint feature space. Models with native heterogeneity support are excluded from this automatic configuration.

Differential Revision: D101841497
facebook#5193)

Summary:

Switches the default heterogeneous transfer learning model from a specialized per-task kernel model to a standard multi-task GP with learned feature imputation. The previous default model class is marked as deprecated.

Differential Revision: D102197137
…ook#5194)

Summary:

Adds per-task outcome standardization to the transfer learning adapter, ensuring each task's observations are standardized independently rather than jointly. Updates the default transform pipeline to use TL-specific outcome transforms.

This removes ambiguity on whether the right transforms have been applied (e.g. QuickBO/warm-starting), where standardization is not performed across, but within experiments.

Differential Revision: D102197139
…ebook#5195)

Summary:

Refactors the learned imputation argument dispatcher to delegate feature index computation to the dataset's built-in mapping utility. This eliminates duplicated feature-ordering logic and ensures consistency with the canonical ordering convention.

Differential Revision: D102197138
@meta-codesync meta-codesync Bot changed the title Use get_heterogeneous_feature_mapping in LFI argparse dispatcher Use get_heterogeneous_feature_mapping in LFI argparse dispatcher (#5195) Apr 29, 2026
@hvarfner hvarfner force-pushed the export-D102197138 branch from 764e169 to 1b629a3 Compare April 29, 2026 14:57
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.38710% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 96.38%. Comparing base (12ebbd9) to head (1b629a3).

Files with missing lines Patch % Lines
ax/adapter/transfer_learning/adapter.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5195      +/-   ##
==========================================
- Coverage   96.38%   96.38%   -0.01%     
==========================================
  Files         617      617              
  Lines       69463    69481      +18     
==========================================
+ Hits        66954    66970      +16     
- Misses       2509     2511       +2     

☔ 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.

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 meta-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants