Skip to content

Conversation

@azurechen97
Copy link
Contributor

@azurechen97 azurechen97 commented Oct 10, 2025

Motivation and Context

There was a formatting issue in the original README.md under section Performance & Limitations:

Q: How can I improve TabPFN’s performance? A: Best practices:

  • Use AutoTabPFNClassifier from TabPFN Extensions for post-hoc ensembling
  • Feature engineering: Add domain-specific features to improve model performance Not effective:
    • Adapt feature scaling
    • Convert categorical features to numerical values (e.g., one-hot encoding)

In which "Not effective" follows "Feature engineering" without a line break, making it confusing.


Public API Changes

  • No Public API changes
  • Yes, Public API changes (Details below)

How Has This Been Tested?


Checklist

  • The changes have been tested locally.
  • Documentation has been updated (if the public API or usage changes).
  • A entry has been added to CHANGELOG.md (if relevant for users).
  • The code follows the project's style guidelines.
  • I have considered the impact of these changes on the public API.

@azurechen97 azurechen97 requested a review from a team as a code owner October 10, 2025 02:49
@azurechen97 azurechen97 requested review from noahho and removed request for a team October 10, 2025 02:49
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request correctly fixes a formatting issue in README.md by adding a blank line. I've also identified a related formatting issue in the same section and provided a suggestion to improve it for better rendering and clarity.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@CLAassistant
Copy link

CLAassistant commented Oct 10, 2025

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@noahho noahho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@noahho noahho merged commit 33d6160 into PriorLabs:main Oct 10, 2025
1 check was pending
oscarkey pushed a commit that referenced this pull request Nov 12, 2025
* Record copied public PR 542

* Fixed a formatting issue in README.md (#542)

Co-authored-by: noahho <noah@priorlabs.ai>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
(cherry picked from commit 33d6160)

---------

Co-authored-by: mirror-bot <mirror-bot@users.noreply.github.com>
Co-authored-by: azurechen97 <36040190+azurechen97@users.noreply.github.com>
Co-authored-by: noahho <noah@priorlabs.ai>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants