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

fix: remove useless token #1765

Merged
merged 1 commit into from
Oct 19, 2023
Merged

fix: remove useless token #1765

merged 1 commit into from
Oct 19, 2023

Conversation

rtrompier
Copy link
Contributor

This token is not used by your action.
Secret is removed from the repository.

@rtrompier rtrompier requested a review from mishig25 October 19, 2023 11:36
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Oct 19, 2023

The documentation is not available anymore as the PR was closed or merged.

Copy link
Contributor

@Wauplin Wauplin left a comment

Choose a reason for hiding this comment

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

Thanks @rtrompier! Merging this now :)

@Wauplin Wauplin merged commit b84a0df into main Oct 19, 2023
@Wauplin Wauplin deleted the rtrompier-patch-1 branch October 19, 2023 12:20
Wauplin added a commit that referenced this pull request Oct 26, 2023
* removed tags from endpoint tests

* removed tags from endpoints

* removed tags from hf_api

* removed tags from docstrings in endpoint_helpers

* removed tags from hf_api

* removed model search argument from test_hf_api

* removed ModelSearchArguments and DataSearchArguments

* removed DatasetSearchArguments and ModelSearchArguments

* removed DatasetSearchArguments and

* removed ModelSearchArguments and DatasetSearchArguments from the docs

* Revert "removed DatasetSearchArguments and"

This reverts commit ce6b91b.

* removed tags from __init__.py

* ran make style

* Removed ## How to explore filter options ? section

* Revert "removed tags from __init__.py"

This reverts commit ad1a31c.

Reverting removal get_dataset_tags and get_model_tags for comment 2

* Revert "removed DatasetSearchArguments and ModelSearchArguments"

This reverts commit fbf6dd0.

* Revert "removed tags from __init__.py"

This reverts commit ad1a31c.

* Revert "removed tags from hf_api"

This reverts commit 2cefee1.

* Revert "removed tags from hf_api"

This reverts commit dd3b8f1.

* Removed attribute dictionary from imports
and removed model search argument class

* Complete removed class AttributeDictionary(dict):

* Removed attribute dictionary tests

* Updating ModelTags and DatasetTags
so that they just return the raw dictionary

* Removed final DatasetTags import

* Removed 'ModelTags' import

* Ran make style

* fix: remove useless token (#1765)

* Retry on ConnectionError/ReadTimeout when streaming file from server (#1766)

* Retry on ConnectionError/ReadTimeout when streaming file from server

* add test

* fix testing utils

* Adding `InferenceClient.get_recommended_model` (#1770)

* Moved logger info to InferenceClient, so get_recommended_model function can bypass that

* Added get_recommended_model to InferenceClient

* Ran make style to generate the async client

* Added tests of get_recommended_model

* Update src/huggingface_hub/inference/_client.py

Co-authored-by: Lucain <[email protected]>

* Fixed ordering of logger info and _get_recommended_model, for model string to have been populated

* Removed _get_recommended_model private function, in favor of get_recommended_model in InferenceClient

* Fixed wording of ValueError to use 'model' not 'task'

* Ran make style for AsyncInferenceClient

---------

Co-authored-by: Lucain <[email protected]>

* Fix document link for manage-cache (#1774)

* Fix document link for manage-cache

* Use redirects in _redirects.yml

* Update docs/source/en/package_reference/file_download.md

---------

Co-authored-by: Lucain <[email protected]>

* Minor doc fixes (#1775)

* Don't use `api` in `list_repo_refs` example.

* Minor typo fssepc -> fsspec

* Use `.item_object_id` instead of `._id`

* Ran make style

---------

Co-authored-by: Remy <[email protected]>
Co-authored-by: Lucain <[email protected]>
Co-authored-by: James Braza <[email protected]>
Co-authored-by: liuxueyang <[email protected]>
Co-authored-by: Pedro Cuenca <[email protected]>
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.

4 participants