skops

skops copied to clipboard

merveenoyan

Remove redundant test in CI

1

As discussed, the "Tests" and "Doc Tests" tasks are redundant. Since CI is already quite slow as is, we really don't want to run all the tests twice. Therefore, the...

BenjaminBossan

Utility function to add arbitrary files to the repo

3

Solves #111 [This](https://huggingface.co/skops-ci/hf_hub_example-c57afd8c-0f9f-4dcc-9fd0-29325a03cf83/tree/main) is the outcome of using `plot_hf_hub.py` with one added line: ```python hub_utils.add_files([__file__], dst=local_repo) ``` As can be seen, the script itself is now included in the upload.

BenjaminBossan

Remove token from hub_utils.tests.common.py and move to secrets

8

We should remove the token from the repo and move it to secrets, but we should make sure it works on PRs from forks.

adrinjalali

Make sure the package is explicitly set as "noarch"

1

This affects platforms/frameworks which only support noarch packages, like pyodide.

adrinjalali

Have labels in config.json for text-classification

10

We need to provide class labels for widget in `config.json` of `text-classification` models. See [this line](https://github.com/huggingface/api-inference-community/blob/main/docker_images/common/app/pipelines/text_classification.py#L19) in `api-inference-community`. (I think it's good to keep it regardless of widget, it can...

merveenoyan

RFC Include generating file(s) in the repo

8

While thinking about https://github.com/skops-dev/skops/issues/110 I realized it'd be much easier to fix existing issues in repos if the file generating the model and the modelcard was included in the repo....

adrinjalali

Figure out CI issues

1

Since we've added inference tests, the CI almost always times out. We should figure out why, starting by putting times in tests to see where the tests fail.

adrinjalali

Add text-regression tasks

4

Similar to #100, but we should also make sure the config file and the README contain the right components for the widget to work. This might also involve adding a...

adrinjalali

Try 4 parallel, remove redundant tests

BenjaminBossan