Skip to content

Fix push to HF hub #54

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

mnickkk
Copy link

@mnickkk mnickkk commented May 27, 2025

tokenizer is undefined, we should use processor from that cell instead:

model, processor = FastModel.from_pretrained(
    model_name = "unsloth/csm-1b",
    max_seq_length= 2048, # Choose any for long context!
    dtype = None, # Leave as None for auto-detection
    auto_model = CsmForConditionalGeneration,
    load_in_4bit = False, # Select True for 4bit - reduces memory usage
)

@mmathew23
Copy link
Contributor

Hi @mnickkk, thank you for the PR and pointing this issue out!

We have some contributing guidelines for the notebooks in the README. Mostly we need to update the template and then update the sesame notebook. If you'd like to try submitting again with the guidelines in mind that would be great.

Otherwise I can submit on top of your PR and then merge into the main repo. Let me know!

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.

2 participants