No more use_auth_token=True #733

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

Merged

patrickvonplaten merged 7 commits into main from remove_use_auth_token

Oct 5, 2022

Contributor

patrickvonplaten commented Oct 5, 2022 •

edited

Loading

Checklist

Bumped huggingface_hub to 0.10.0
Removed all use_auth_token=True.
use_auth_token=use_auth_token was kept since people might want to not be logged in and pass specific tokens

TODO until release

Update all official colabs and blog posts in the open

patrickvonplaten added 4 commits

October 5, 2022 13:42

up

d7bdbf2

uP

396f289

uP

99b1d4b


          make style

c5fb0b1

patrickvonplaten commented

View reviewed changes

README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

docs/source/optimization/fp16.mdx Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

docs/source/optimization/fp16.mdx Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

docs/source/optimization/fp16.mdx Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

docs/source/optimization/fp16.mdx Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

src/diffusers/configuration_utils.py Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

src/diffusers/modeling_flax_utils.py Outdated Show resolved Hide resolved

HuggingFaceDocBuilderDev commented Oct 5, 2022

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

patrickvonplaten commented

View reviewed changes

src/diffusers/modeling_utils.py Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

src/diffusers/pipelines/README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

src/diffusers/pipelines/README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

src/diffusers/pipelines/stable_diffusion/README.md Outdated Show resolved Hide resolved

patrickvonplaten commented

View reviewed changes

src/diffusers/pipelines/stable_diffusion/README.md Outdated Show resolved Hide resolved

patrickvonplaten added 3 commits

October 5, 2022 15:50


          Apply suggestions from code review

e347b26

up

d4247cc


          finish

ce9edee

patrickvonplaten requested review from anton-l, patil-suraj and pcuenca

October 5, 2022 14:14

patil-suraj approved these changes

View reviewed changes

Contributor

patil-suraj left a comment

Thanks a lot ! We should also remove this from the textutal_inversion and dreambooth PR

patrickvonplaten merged commit 78744b6 into main

patrickvonplaten deleted the remove_use_auth_token branch

October 5, 2022 15:16

pcuenca approved these changes

View reviewed changes

Member

pcuenca left a comment

Awesome!

src/diffusers/dynamic_modules_utils.py

    
                  <Tip>

                  Passing `use_auth_token=True` is required when you want to use a private model.

                  Passing `` is required when you want to use a private model.

Member

pcuenca Oct 5, 2022

Suggested change

      
                Passing `` is required when you want to use a private model.
          
                You may pass a token in `use_auth_token` if you are not logged in (`huggingface-cli long`) and
          
                want to use private or [gated models](https://huggingface.co/docs/hub/models-gated#gated-models).

src/diffusers/dynamic_modules_utils.py

    
                  <Tip>

                  Passing `use_auth_token=True` is required when you want to use a private model.

                  Passing `` is required when you want to use a private model.

Member

pcuenca Oct 5, 2022

Suggested change

      
                Passing `` is required when you want to use a private model.
          
                You may pass a token in `use_auth_token` if you are not logged in (`huggingface-cli long`) and
          
                want to use private or [gated models](https://huggingface.co/docs/hub/models-gated#gated-models).

pcuenca added a commit that referenced this pull request


          Replace message with empty backquotes.

753bccf

This was part of #733, I was too slow to review :)

pcuenca mentioned this pull request

Replace messages that have empty backquotes #738

Merged

pcuenca added a commit that referenced this pull request


          Replace messages that have empty backquotes (#738)

Replace message with empty backquotes.

This was part of #733, I was too slow to review :)

shirayu added a commit to shirayu/purepale that referenced this pull request


          Removed "use_auth_token" huggingface/diffusers#733

7189c24

prathikr pushed a commit to prathikr/diffusers that referenced this pull request


          No more use_auth_token=True (huggingface#733)

23978d1

* up

* uP

* uP

* make style

* Apply suggestions from code review

* up

* finish

prathikr pushed a commit to prathikr/diffusers that referenced this pull request


          Replace messages that have empty backquotes (huggingface#738)

6a28ab3

Replace message with empty backquotes.

This was part of huggingface#733, I was too slow to review :)

yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request


          No more use_auth_token=True (huggingface#733)

54bc44b

* up

* uP

* uP

* make style

* Apply suggestions from code review

* up

* finish

yoonseokjin pushed a commit to yoonseokjin/diffusers that referenced this pull request


          Replace messages that have empty backquotes (huggingface#738)

c369b60

Replace message with empty backquotes.

This was part of huggingface#733, I was too slow to review :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet