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.

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

allow passing hf_token to load_textual_inversion #10546

Merged

Conversation

Teriks
Copy link
Contributor

@Teriks Teriks commented Jan 13, 2025

Allows passing HF_TOKEN to load_textual_inversion explicitly using the argument hf_token as suggested in the linked issue.

Fixes duplicate parameter name conflict caused by kwargs usage, as token is used by this method to denote a lexical token and not an authentication token.

Fixes #7689

Before submitting

Who can review?

@sayakpaul

@sayakpaul
Copy link
Member

Cc: @yiyixuxu WDYT?

Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

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

Thanks!

@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.

@yiyixuxu
Copy link
Collaborator

thanks! can we update the docstring here accordingly?

token (`str` or *bool*, *optional*):

@Teriks Teriks force-pushed the load_textual_inversion_hf_token_fix branch from f31c0a8 to 583164a Compare January 13, 2025 21:53
@Teriks
Copy link
Contributor Author

Teriks commented Jan 13, 2025

thanks! can we update the docstring here accordingly?

token (`str` or *bool*, *optional*):

synced fork and double checked that token is now hf_token in the docstring

on line 315 of the diff

doc: https://moon-ci-docs.huggingface.co/docs/diffusers/pr_10546/en/api/loaders/textual_inversion#diffusers.loaders.TextualInversionLoaderMixin.load_textual_inversion.hf_token

@yiyixuxu yiyixuxu merged commit 6b72784 into huggingface:main Jan 14, 2025
12 checks passed
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.

Auth token is not passable to load_textual_inversion due to named argument conflicting with accepted kwargs
4 participants