Lucain

[WIP] enable coverage in CI

1

Related to https://github.com/huggingface/huggingface_hub/issues/977. Draft mode since it's not close to be reviewed :D

[WIP] First proposal for a utility to scan cache

2

Following issue https://github.com/huggingface/huggingface_hub/issues/972. This is a draft of what the scan of the cache directory could look like. At the moment there is only a `scan_cache` method that returns structured...

[WIP] enable pytest-xdist in CI

2

Related to https://github.com/huggingface/huggingface_hub/issues/970. Draft mode since it's not close to be reviewed :D

Enable code coverage properly in Github Actions

1

Following up on PR https://github.com/huggingface/huggingface_hub/pull/976 that disables the coverage in tests. It would be good to re-enable and fix it properly. Potential fixes have been mentioned [here (internal url)](https://huggingface.slack.com/archives/C02V5EA0A95/p1657176912893549?thread_ts=1657099388.126399&cid=C02V5EA0A95) and...

Speed-up tests using `pytest-xdist`

5

Github CI for `huggingface_hub` is doing a lot of good things but I feel tests speed could be improved to get a shorter feedback loop on PRs. An idea to...

Handle redirections in `hf_hub_download` for a renamed repo

7

Fix https://github.com/huggingface/huggingface_hub/issues/981. Solution I went for is to force redirection only when the `Location` header is relative which is the case for renamed repos. It doesn't seem that an out-of-the-box...

In `test_push_to_hub_model_kwargs` and `test_save_pretrained_kwargs_load_fails_without_traces`, it seems that the tests are not working as expected. They should apparently check that loading from pretrained weights raises an exception if custom layers are...

*Originally from @LysandreJik [in this comment](https://github.com/huggingface/huggingface_hub/pull/916#issuecomment-1196419984)* > One thing I would maybe clarify in the docs is the purpose of the `parent_commit` parameter. I understand its purpose easily in the...

*Originally mentioned by @sgugger [on slack (internal url)](https://huggingface.slack.com/archives/C023JAKTR2P/p1659446501418569) with reproducible example:* > Hi there! There is a little bug with repo that are moved by users, it seems that while...

Make `path_in_repo` optional in `upload_folder`

1

At the moment, `path_in_repo` argument is required in `upload_folder` method. This makes it weird when a user wants to upload a folder to the root of the repo. Example: ```py...