Skip to content

Conversation

@HendrikStrobelt
Copy link
Contributor

No description provided.

@mergify
Copy link

mergify bot commented Oct 2, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert|release)(?:\(.+\))?:

@HendrikStrobelt HendrikStrobelt changed the title making Granite 4 the default feat: making Granite 4 the default Oct 2, 2025
# Conflicts:
#	docs/examples/information_extraction/101_with_gen_slots.py
#	docs/examples/mini_researcher/researcher.py
@HendrikStrobelt HendrikStrobelt marked this pull request as ready for review October 2, 2025 23:20
@HendrikStrobelt HendrikStrobelt changed the title feat: making Granite 4 the default feat: making Granite 4 the default model Oct 2, 2025
Copy link
Contributor

@jakelorocco jakelorocco left a comment

Choose a reason for hiding this comment

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

lgtm

Comment on lines +31 to +32
watsonx_name="ibm/granite-4-h-small",
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we care that if we eventually switch this we might change results / performance for watsonx backend users? I don't think so, just wanted to flag that.

@HendrikStrobelt HendrikStrobelt merged commit 545c1b3 into main Oct 3, 2025
4 checks passed
@HendrikStrobelt HendrikStrobelt deleted the hen/g4 branch October 3, 2025 14:27
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