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

Unpin hfh #6876

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

Unpin hfh #6876

wants to merge 1 commit into from

Conversation

lhoestq
Copy link
Member

@lhoestq lhoestq commented May 6, 2024

Needed to use those in dataset-viewer:

close #6863

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

Copy link
Member

@albertvillanova albertvillanova left a comment

Choose a reason for hiding this comment

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

As I explain in the corresponding issue that I self-assigned (#6863), I was planning to unpin the upper bound of huggingface-hub < 0.23.0 only when the huggingface/transformers#30618 fix is merged and released in a new transformers version, otherwise it breaks our CI. The fix has been merged but not released yet.

Also note that datasets-2.19.1 (version currently installed in the dataset-viewer) does not include the pin in huggingface-hub:
https://github.com/huggingface/datasets/blob/2.19.1/setup.py#L138

"huggingface-hub>=0.21.2",

If we urgently need some dev feature for dataset-viewer, I would suggest pushing the feature (cherry-picked) to a dedicated branch with 2.19.1 as its starting point (without opening a PR), and install datasets from that branch.

@lhoestq
Copy link
Member Author

lhoestq commented May 7, 2024

transformers 4.40.2 was release yesterday but not sure if it contains the fix

@albertvillanova
Copy link
Member

albertvillanova commented May 7, 2024

@lhoestq yes I knew transformers 4.40.2 was released yesterday, but I had checked that it does not contain the fix: only 2 bug fixes. That is why our CI continues failing in this PR. We will have to wait until the next minor version.

@albertvillanova
Copy link
Member

If we urgently need some dev feature for dataset-viewer, I would suggest pushing the feature (cherry-picked) to a dedicated branch with 2.19.1 as its starting point (without opening a PR), and install datasets from that branch.

I have done so:

@lhoestq
Copy link
Member Author

lhoestq commented May 22, 2024

hfh 0.23.1 and transformers 4.41.0 as are out out, let's unpin no ?

@albertvillanova
Copy link
Member

I have re-run the CI to check that is green before.

Copy link
Member

@albertvillanova albertvillanova left a comment

Choose a reason for hiding this comment

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

The CI is red.

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.

Revert temporary pin huggingface-hub < 0.23.0
4 participants