Skip to content

fix issue #87: completing tabicl missing documentation - #94

Open
4xel-C wants to merge 8 commits into
Bayer-Group:mainfrom
4xel-C:fix/issue-87-documentation-tabicl
Open

4xel-C wants to merge 8 commits into
Bayer-Group:mainfrom
4xel-C:fix/issue-87-documentation-tabicl

Conversation

@4xel-C

@4xel-C 4xel-C commented Oct 1, 2026

Copy link
Copy Markdown
Contributor

Description

This PR updates the MotherML documentation to include TabICL support, installation instructions, and the associated example notebooks.

Files changed

README.md

  • Added the tabicl optional extra with tabicl and torch dependencies.
  • Added TabICL and SHAP explainability to the optional dependencies table.
  • Updated pip installation examples to include the tabicl extra.
  • Updated uv installation examples to include the tabicl extra.

docs/development/dev.md

  • Added tabicl to the optional extras used for the development environment.
  • Added the tabicl extra to the extras documentation.
  • Updated the multiple-extras installation example.

docs/examples.md

  • Added the TabICL regression notebook.
  • Added the TabICL classification notebook.
  • Added the TabICL embedding transformer notebook for embedding extraction.

docs/mother/models.md

  • Added tabicl to the list of built-in algorithm groups.
  • Added TabICLRegressorMother and TabICLClassifierMother to the supported models table.

examples/notebooks/README.md

  • Added the TabICL regression example with built-in SHAP explainability.
  • Added the TabICL classification example with built-in SHAP explainability.
  • Added the TabICL embedding extraction example.

Closes #87

Copilot AI balanced review requested due to automatic review settings October 1, 2026 12:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The documentation fulfills the issue’s acceptance criteria; only two minor wording corrections remain.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

Documents existing TabICL support and its tutorials across user and developer documentation.

Changes:

  • Adds TabICL installation and model references.
  • Links regression, classification, and embedding tutorials.
  • Documents built-in SHAP support.
File Description
README.md Adds the TabICL extra and installation examples.
docs/​development/​dev.md Documents TabICL development installation.
docs/​examples.md Links the three TabICL tutorials.
docs/​mother/​models.md Lists TabICL models.
examples/​notebooks/​README.md Adds TabICL notebooks to the catalog.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread examples/notebooks/README.md Outdated
…er-Group#87)

Corrected capitalization and formatting in README for TabICL model description.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 1, 2026 13:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The acceptance criteria are satisfied; only a minor wording correction remains.

Review effort: Balanced
Findings: None

Resolved since last review (1)
Previously missed (1)

In code that hasn't changed since last review

Low severity Correct capitalization and change “build-in” to “built-in”

examples/​notebooks/​README.md:64

Use the product’s established capitalization and correct “build-in” to “built-in” in this notebook description.

Copilot AI balanced review requested due to automatic review settings October 1, 2026 14:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The documentation meets the issue’s acceptance criteria, with only a minor wording correction noted.

Review effort: Balanced
Findings: 1 Low severity

Open (1)

Comment thread examples/notebooks/README.md
@4xel-C
4xel-C force-pushed the fix/issue-87-documentation-tabicl branch from e783ea4 to 616cfaa Compare October 1, 2026 15:19
Copilot AI balanced review requested due to automatic review settings October 5, 2026 09:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

The linked issue’s embedding utility documentation remains incomplete, and one new notebook description contains a typo.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
Resolved since last review (1)

Comment thread docs/mother/models.md Outdated
Copilot AI balanced review requested due to automatic review settings October 5, 2026 12:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The acceptance criteria are satisfied; only minor documentation typos remain.

Review effort: Balanced
Findings: 1 Low severity

Open (1)
Resolved since last review (1)

Comment thread docs/mother/models.md

This branch has not been deployed

No deployments
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.

Document TabICL support

2 participants