Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[transformers][FLCE] make compatible with latest (>=4.49.0) XXXForCausalLM.forward APIs #573

Open
yundai424 opened this issue Feb 20, 2025 · 0 comments · May be fixed by #596
Open

[transformers][FLCE] make compatible with latest (>=4.49.0) XXXForCausalLM.forward APIs #573

yundai424 opened this issue Feb 20, 2025 · 0 comments · May be fixed by #596
Labels
good first issue Good for newcomers huggingface

Comments

@yundai424
Copy link
Collaborator

yundai424 commented Feb 20, 2025 •
edited
Loading

🐛 Describe the bug

Lately in this commit huggingface/transformers#35757 HF slightly changed the args to XXXForCausalLM.forward method. We should make our FLCE compatible with the latest version

Reproduce

No response

Versions

transformers==4.48.3
@yundai424 yundai424 changed the title [transformers][FLCE] make compatible with latest (>=4.48) XXXForCausalLM.forward APIs [transformers][FLCE] make compatible with latest (>=4.49.0) XXXForCausalLM.forward APIs Feb 20, 2025
This was referenced Mar 5, 2025
Draft
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers huggingface
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Make FLCE compatible with latest XXXForCausalLM.forward() APIs linkedin/Liger-Kernel
1 participant
@yundai424