Rename ModelInterfaceConfig to InferenceConfig. #575
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Also move it from `tabpfn.config` to `tabpfn.inference_config`, to distinguish this module from all the other config modules we have. We already refer to it as the "inference config" in the classifier/regressor interfaces, so this helps to be consistent.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Code Review
This pull request refactors ModelInterfaceConfig to InferenceConfig and moves it to a new module tabpfn.inference_config. The changes are consistent and well-executed across the codebase. I've found a couple of minor issues: a placeholder in the changelog and an outdated docstring that should be updated to reflect the renaming.
bejaeger
approved these changes
Oct 29, 2025
Contributor
bejaeger
left a comment
bejaeger
left a comment
There was a problem hiding this comment.
Very nice! Thanks for changing this.
oscarkey
added a commit
to PriorLabs/tabpfn-client
that referenced
this pull request
Oct 30, 2025
The name of this class was changed in tabpfn in PriorLabs/TabPFN#575
oscarkey
added a commit
to PriorLabs/tabpfn-extensions
that referenced
this pull request
Oct 31, 2025
ModelInterfaceConfig was renamed and moved in PriorLabs/TabPFN#575 . This change works with both versions.
oscarkey
added a commit
that referenced
this pull request
Nov 12, 2025
) * Record copied public PR 575 * Rename ModelInterfaceConfig to InferenceConfig. (#575) Also move it from `tabpfn.config` to `tabpfn.inference_config`, to distinguish this module from all the other config modules we have. We already refer to it as the "inference config" in the classifier/regressor interfaces, so this helps to be consistent. This changes the public api: - code searches ([1](https://github.com/search?q=org%3APriorLabs+ModelInterfaceConfig+NOT+repo%3APriorLabs%2FTabPFN+NOT+repo%3APriorLabs%2FTabPFN-private+&type=code), [2](https://github.com/search?q=org%3APriorLabs+interface_config_+NOT+repo%3APriorLabs%2FTabPFN+NOT+repo%3APriorLabs%2FTabPFN-private+&type=code)) show one use in tabpfn-extensions and in the tabpfn-client documentation. I will fix these - this could break external users, but as it's a major version upgrade, an easy fix, and probably few people are using it anyway, a quick offline discussion with Noah concluded it's okay --------- Co-authored-by: mirror-bot <mirror-bot@users.noreply.github.com> Co-authored-by: Oscar Key <oscar@priorlabs.ai>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Also move it from
tabpfn.configtotabpfn.inference_config, to distinguish this module from all the other config modules we have.We already refer to it as the "inference config" in the classifier/regressor interfaces, so this helps to be consistent.
This changes the public api: