Skip to content

TabPFN updates#1598

Merged
bgruening merged 19 commits intobgruening:masterfrom
anuprulez:tabpfn_license
Mar 26, 2025
Merged

TabPFN updates#1598
bgruening merged 19 commits intobgruening:masterfrom
anuprulez:tabpfn_license

Conversation

@anuprulez
Copy link
Copy Markdown
Contributor

The PR adds:

  • Appropriate license (in the help section)
  • Comparison to CatBoost approach for classification and regression

@@ -1,119 +1,123 @@
<?xml version="1.0"?>
<tool id="tabpfn" name="Tabular data prediction using TabPFN" version="@TOOL_VERSION@+galaxy@VERSION_SUFFIX@" profile="23.0">
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Do we need to change the name now?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

<?xml version="1.0"?> was introduced by xmllint. It is removed now.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I agree if the tool name is not updated while keeping CatBoost comparison, it may not sound good.

anuprulez and others added 6 commits March 26, 2025 13:18
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
Co-authored-by: Björn Grüning <bjoern@gruenings.eu>
@bgruening bgruening enabled auto-merge March 26, 2025 16:02
@bgruening bgruening merged commit cefdfdc into bgruening:master Mar 26, 2025
11 checks passed
@anuprulez anuprulez deleted the tabpfn_license branch March 26, 2025 16:24
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.

2 participants