Skip to content

fix: disable granite in custom gptq as gptqmodel already supports it, fix …#130

Merged
chichun-charlie-liu merged 1 commit intofoundation-model-stack:mainfrom
chichun-charlie-liu:fix4PR102
Jun 5, 2025
Merged

fix: disable granite in custom gptq as gptqmodel already supports it, fix …#130
chichun-charlie-liu merged 1 commit intofoundation-model-stack:mainfrom
chichun-charlie-liu:fix4PR102

Conversation

@chichun-charlie-liu
Copy link
Copy Markdown
Collaborator

…for PR102

Description of the change

minor fix for previously merged PR 102, just disable granite in custom gptq model as gptqmodel package has native support already.

Related issues or PRs

#102

(Optional) List any documentation or testing added

(Optional) How to verify the contribution

Checklist for passing CI/CD:

  • All commits are signed showing "Signed-off-by: Name <email@domain.com>" with git commit -signoff or equivalent
  • PR title and commit messages adhere to Conventional Commits
  • Contribution is formatted with tox -e fix
  • Contribution passes linting with tox -e lint
  • Contribution passes spellcheck with tox -e spellcheck
  • Contribution passes all unit tests with tox -e unit

Note: CI/CD performs unit tests on multiple versions of Python from a fresh install. There may be differences with your local environment and the test environment.

…for PR102

Signed-off-by: cliu-us <cliu@us.ibm.com>
Copy link
Copy Markdown
Collaborator

@tharapalanivel tharapalanivel left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @chichun-charlie-liu

@chichun-charlie-liu chichun-charlie-liu merged commit b2a3c85 into foundation-model-stack:main Jun 5, 2025
11 checks passed
@chichun-charlie-liu chichun-charlie-liu deleted the fix4PR102 branch June 5, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants