Nightly by danielhanchen · Pull Request #3767 · unslothai/unsloth

danielhanchen · 2025-12-23T12:50:40Z

No description provided.

for more information, see https://pre-commit.ci

@danielhanchen

* Fix get_input_embeds call for VLMs * patch input_require_grads instead * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * cleanup old patch * cleanup old patch * cleanup * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply suggestion from @danielhanchen * use logger instead of prints * Move unsloth present set * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Daniel Han <danielhanchen@gmail.com>

for more information, see https://pre-commit.ci

…ightly

for more information, see https://pre-commit.ci

…ightly

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Silence fbgemm TMA print Also safer .push_to_hub * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

…ightly

for more information, see https://pre-commit.ci

@danielhanchen

* login on token * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * cleanup old code * safer imports * cleanup * Return token after login * correct return types * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply suggestion from @danielhanchen * add back imports * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * finish return token --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Daniel Han <danielhanchen@gmail.com>

* Do not overwrite slots * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

gemini-code-assist · 2025-12-23T12:50:53Z

Summary of Changes

Hello @danielhanchen, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request refines the automatically generated README content for models pushed to the Hugging Face Hub using Unsloth. It simplifies the attribution for merged models, updates the title format and adds Unsloth branding to GGUF model READMEs, and provides more current example usage commands for GGUF models with llama.cpp.

Highlights

Streamlined Attribution: Removed the mention of Huggingface's TRL library from the README content generated for merged models, focusing solely on Unsloth attribution.
GGUF README Enhancements: Updated the title format for GGUF model READMEs and added prominent Unsloth training attribution and logo.
Updated GGUF Usage Examples: Revised the example commands for running GGUF models, now using the -hf and --jinja flags with llama.cpp.

🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	`/gemini review`	Performs a code review for the current pull request in its current state.
Pull Request Summary	`/gemini summary`	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	`/gemini help`	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

for more information, see https://pre-commit.ci

gemini-code-assist

Code Review

This pull request updates the content of model cards and README templates. The changes simplify some messages and improve the example usage instructions for GGUF models. My review identifies a minor code duplication in the promotional footer being added, which could be refactored to improve maintainability.

gemini-code-assist · 2025-12-23T12:52:51Z

+        readme_content += (
+            "This was trained 2x faster with [Unsloth](https://github.com/unslothai/unsloth)\n"
+            '[<img src="https://raw.githubusercontent.com/unslothai/unsloth/main/images/unsloth%20made%20with%20love.png" width="200"/>](https://github.com/unslothai/unsloth)\n'
+        )


This promotional footer is very similar to the one in MODEL_CARD (lines 1432-1434). To improve maintainability, consider abstracting the duplicated image markdown into a constant. This would make future updates to branding assets easier and less error-prone.

@danielhanchen

* Update _utils.py * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * [FIX] [Transformers] VLM input embeds fix for gradients (unslothai#3715) * Fix get_input_embeds call for VLMs * patch input_require_grads instead * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * cleanup old patch * cleanup old patch * cleanup * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply suggestion from @danielhanchen * use logger instead of prints * Move unsloth present set * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Daniel Han <danielhanchen@gmail.com> * Update rope_embedding.py * Fixes * Update _utils.py * Update import_fixes.py * Update rl_replacements.py * fix_openenv_no_vllm * Fix * Update __init__.py * Update __init__.py * Update __init__.py * Update import_fixes.py * Update import_fixes.py * Update import_fixes.py * logger * Update __init__.py * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update __init__.py * Update import_fixes.py * Update __init__.py * Update import_fixes.py * Update import_fixes.py * Update import_fixes.py * Update import_fixes.py * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Update import_fixes.py * Update unsloth/import_fixes.py Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com> * Update save.py * [fbgemm] Silence tma fbgemm (unslothai#3735) * Silence fbgemm TMA print Also safer .push_to_hub * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> * Update loader.py * Update save.py * Update save.py * Update _utils.py * Update _utils.py * Diffusers warnings * Update pyproject.toml * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * [hf_hub] Token login (unslothai#3739) * login on token * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * cleanup old code * safer imports * cleanup * Return token after login * correct return types * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * Apply suggestion from @danielhanchen * add back imports * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * finish return token --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Daniel Han <danielhanchen@gmail.com> * Do not overwrite slots (unslothai#3752) * Do not overwrite slots * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> * Update save.py * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Datta Nimmaturi <venkatadattasainimmaturi@gmail.com> Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

danielhanchen and others added 30 commits December 9, 2025 01:02

Update _utils.py

e53e185

Merge branch 'main' into nightly

150eadf

Merge branch 'main' into nightly

25a6250

Merge branch 'main' into nightly

6b908cf

Merge branch 'main' into nightly

f754bd2

[pre-commit.ci] auto fixes from pre-commit.com hooks

30ade52

for more information, see https://pre-commit.ci

Merge branch 'main' into nightly

94bbcaa

Merge branch 'main' into nightly

30b78c5

Merge branch 'main' into nightly

f357fc5

Merge branch 'main' into nightly

a63a337

Update rope_embedding.py

2c22ce6

Merge branch 'main' into nightly

449430d

Fixes

b5f1a77

Update _utils.py

c94f595

Update import_fixes.py

01319d3

Update rl_replacements.py

696a540

fix_openenv_no_vllm

ac54d6e

Fix

9a98139

Update __init__.py

680f19f

Update __init__.py

fb763f3

Update __init__.py

f4f2a7f

Update import_fixes.py

e17b62f

Update import_fixes.py

f34eb0a

Update import_fixes.py

04ad21c

logger

32b52a0

Update __init__.py

0c9288f

[pre-commit.ci] auto fixes from pre-commit.com hooks

b5b57b3

for more information, see https://pre-commit.ci

Update __init__.py

efb5801

Merge branch 'nightly' of https://github.com/unslothai/unsloth into n…

12ccc47

…ightly

pre-commit-ci Bot and others added 25 commits December 17, 2025 04:26

[pre-commit.ci] auto fixes from pre-commit.com hooks

ca77798

for more information, see https://pre-commit.ci

Update import_fixes.py

71c5938

Merge branch 'nightly' of https://github.com/unslothai/unsloth into n…

2991a7d

…ightly

Update unsloth/import_fixes.py

7c10195

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Merge branch 'main' into nightly

7f744d9

Merge branch 'main' into nightly

d96ac97

Update save.py

3738db7

Merge branch 'nightly' of https://github.com/unslothai/unsloth into n…

6e5a561

…ightly

Update loader.py

3e6cfb5

Update save.py

30a454c

Update save.py

b58663a

Update _utils.py

7b61375

Update _utils.py

a2b5def

Diffusers warnings

59a1fa5

Update pyproject.toml

5e33a07

[pre-commit.ci] auto fixes from pre-commit.com hooks

1f1bf49

for more information, see https://pre-commit.ci

Merge branch 'main' into nightly

0ed6738

Merge branch 'main' into nightly

6346d17

Do not overwrite slots (#3752)

1de77bf

* Do not overwrite slots * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Merge branch 'main' into nightly

4d99c57

Merge branch 'main' into nightly

2d10783

Merge branch 'main' into nightly

7561eee

Update save.py

82df967

[pre-commit.ci] auto fixes from pre-commit.com hooks

0f60650

for more information, see https://pre-commit.ci

danielhanchen merged commit 65f3b4b into main Dec 23, 2025
4 checks passed

gemini-code-assist Bot reviewed Dec 23, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Nightly#3767

Nightly#3767
danielhanchen merged 64 commits into
mainfrom
nightly

danielhanchen commented Dec 23, 2025

Uh oh!

gemini-code-assist Bot commented Dec 23, 2025

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

gemini-code-assist Bot Dec 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Uh oh!

Conversation

danielhanchen commented Dec 23, 2025

Uh oh!

gemini-code-assist Bot commented Dec 23, 2025

Summary of Changes

Highlights

Footnotes

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Dec 23, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants