Skip to content

fix(examples): migrate basics_colab and agents_colab notebooks from granite3.3 to granite4.1:8b#1539

Closed
BiswasNehaa wants to merge 6 commits into
i-am-bee:mainfrom
BiswasNehaa:fix/migrate-granite-3-to-4
Closed

fix(examples): migrate basics_colab and agents_colab notebooks from granite3.3 to granite4.1:8b#1539
BiswasNehaa wants to merge 6 commits into
i-am-bee:mainfrom
BiswasNehaa:fix/migrate-granite-3-to-4

Conversation

@BiswasNehaa

Copy link
Copy Markdown
Contributor

Closes #1481

Migrates the remaining notebook examples referencing granite3.3 to granite4.1:8b:

  • python/examples/notebooks/basics_colab.ipynb
  • python/examples/notebooks/agents_colab.ipynb

basics.ipynb, workflows.ipynb, and workflows_colab.ipynb were already migrated in earlier PRs (#1521, #1534), so this should cover the remaining notebooks listed in the issue.

BiswasNehaa and others added 6 commits June 20, 2026 12:55
@BiswasNehaa
BiswasNehaa requested review from a team as code owners June 30, 2026 17:21
@github-actions github-actions Bot added documentation Improvements or additions to documentation python Python related functionality typescript Typescript related functionality labels Jun 30, 2026
@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jun 30, 2026

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request updates the default Ollama model version from granite3.3:8b to granite4.1:8b across various documentation files, examples, and Jupyter notebooks. Feedback is provided regarding an inconsistency in basics.ipynb where the model name is specified as ollama:granite4.1 instead of ollama:granite4.1:8b, which could lead to runtime errors if the exact tag is not found.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

"# Create a ChatModel to interface with granite3.3:8b on a local ollama\n",
"model = ChatModel.from_name(\"ollama:granite3.3\")\n",
"# Create a ChatModel to interface with granite4.1:8b on a local ollama\n",
"model = ChatModel.from_name(\"ollama:granite4.1\")\n",

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.

medium

The model name is specified as ollama:granite4.1 here, but the pulled model is granite4.1:8b (as shown in the comment above and other notebooks). Using ollama:granite4.1 will cause a runtime error in Ollama if the model is not found under that exact tag. It should be updated to ollama:granite4.1:8b for consistency and correctness.

Suggested change
"model = ChatModel.from_name(\"ollama:granite4.1\")\n",
"model = ChatModel.from_name(\"ollama:granite4.1:8b\")\n",

@BiswasNehaa

Copy link
Copy Markdown
Contributor Author

Closing this in favor of a cleaner PR — this branch picked up some unrelated commits
from a merge mix-up. Opening a fresh PR with just the notebook migration commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation python Python related functionality size:M This PR changes 30-99 lines, ignoring generated files. typescript Typescript related functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate examples from granite3.3:8b to granite4.1:8b

1 participant