Skip to content
New issue

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

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

Already on GitHub? # to your account

notebook_login errors out after passing token in main branch #1072

Closed
merveenoyan opened this issue Sep 22, 2022 · 8 comments · Fixed by #1073
Closed

notebook_login errors out after passing token in main branch #1072

merveenoyan opened this issue Sep 22, 2022 · 8 comments · Fixed by #1073
Labels
bug Something isn't working

Comments

@merveenoyan
Copy link
Contributor

merveenoyan commented Sep 22, 2022

Describe the bug

I installed main branch of huggingface_hub because I needed to use dataset conversion tool where you can see huggingface_hub main branch in requirements.txt. The notebook_login errors out with logs below. This doesn't happen when I update last version of the library.

Reproduction

You can try this notebook. I tried only installing github repository itself and calling notebook_login and was able to reproduce it.

Logs

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/usr/local/lib/python3.7/dist-packages/huggingface_hub/commands/user.py in login_token_event(t)
    308         token_widget.value = ""
    309         clear_output()
--> 310         _login(token=token)
    311 
    312     token_finish_button.on_click(login_token_event)

TypeError: _login() missing 1 required positional argument: 'hf_api'

System Info

(Below is tip of main branch)

huggingface-hub @ git+https://github.com/huggingface/huggingface_hub@3a59bf13f1ef50b2e7e9ac8362f5984ac288eb33
@Wauplin
Copy link
Contributor

Wauplin commented Sep 22, 2022

Hi @merveenoyan , thanks for reporting ! This regression has been introduced by #1064. I fixed it here: #1073. Sorry for that but very happy you noticed it before any release :D

@Wauplin
Copy link
Contributor

Wauplin commented Sep 22, 2022

In the meantime, you can pip install from 26819c73a47e56a53a2e047d540c0c456d31d70c

@merveenoyan
Copy link
Contributor Author

@Wauplin the last release works just fine, it's just the tool I mentioned in the issue is using main branch and that was my initial problem. If I revert to latest release the tool just won't work, that's why I didn't want to use that branch.

@Wauplin
Copy link
Contributor

Wauplin commented Sep 22, 2022

Merged :)

@nateraw
Copy link
Contributor

nateraw commented Sep 26, 2022

@merveenoyan Did installing the latest release solve the problem for you while using the datasets converter? (v0.10.0rc0)

It did not solve the problem for me, I think, which is concerning.

@Wauplin
Copy link
Contributor

Wauplin commented Sep 27, 2022

@nateraw I haven't used the datasets converter tool but here is a simple colab where notebook_login works correctly on 0.10.0rc0. Could you check huggingface_hub is correctly installed ?

pip uninstall huggingface_hub
pip install huggingface_hub==0.10.0rc0

@osanseviero
Copy link
Contributor

The login does work for me when installing from head

@nateraw
Copy link
Contributor

nateraw commented Sep 27, 2022

The issue was fixed with this PR I believe. Thanks all!!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants