KEMBAR78
deci gguf support by ved1beta · Pull Request #38669 · huggingface/transformers · GitHub
Skip to content

Conversation

@ved1beta
Copy link
Contributor

@ved1beta ved1beta commented Jun 8, 2025

What does this PR do?

GGUF support for deci
Fixes #37736

Before submitting

  • Was this discussed/approved via a Github issue or the Please add a link

Who can review?

@MekkCyber

@ved1beta ved1beta closed this Jun 8, 2025
@ved1beta ved1beta deleted the deci_gguf branch June 8, 2025 06:06
@ved1beta ved1beta restored the deci_gguf branch June 8, 2025 06:09
@ved1beta ved1beta reopened this Jun 8, 2025
Copy link
Contributor

@MekkCyber MekkCyber 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 for adding this 🔥 ! can you please add some tests in tests/quantization/ggml/ ?

@MekkCyber MekkCyber requested a review from Isotr0py June 11, 2025 12:43
Comment on lines 983 to 992
try:
original_model = AutoModelForCausalLM.from_pretrained(
original_model_id,
torch_dtype=torch.float16,
trust_remote_code=True,
device_map="auto",
)
except Exception as e:
self.skipTest(f"Original Deci model not available for comparison: {e}")
return
Copy link
Collaborator

@Isotr0py Isotr0py Jun 12, 2025

Choose a reason for hiding this comment

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

Why do we need this try: ... excetpt: .... statement in test? Should't the test failed if it encounter an Exception?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you address this too?

@ved1beta
Copy link
Contributor Author

hey , any update on this ?

@ctcanbol
Copy link
Contributor

hey , any update on this ?

I think after resolving conflicts we're good to go? Looking forward to this pr!

Thanks.

Copy link
Collaborator

@Isotr0py Isotr0py left a comment

Choose a reason for hiding this comment

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

Sorry for missing this thread! LGTM

Comment on lines 983 to 992
try:
original_model = AutoModelForCausalLM.from_pretrained(
original_model_id,
torch_dtype=torch.float16,
trust_remote_code=True,
device_map="auto",
)
except Exception as e:
self.skipTest(f"Original Deci model not available for comparison: {e}")
return
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you address this too?

@ved1beta
Copy link
Contributor Author

thanks for keeping up with the pr ❤️

@Isotr0py Isotr0py enabled auto-merge (squash) July 29, 2025 04:42
@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Copy link
Member

@SunMarc SunMarc left a comment

Choose a reason for hiding this comment

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

LGTM !

@github-actions
Copy link
Contributor

[For maintainers] Suggested jobs to run (before merge)

run-slow: ggml

@Isotr0py Isotr0py merged commit 0ce6709 into huggingface:main Aug 26, 2025
24 checks passed
@ved1beta ved1beta deleted the deci_gguf branch August 26, 2025 14:28
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.

ValueError: GGUF model with architecture deci is not supported yet.

6 participants