Rename k2_sherpa to k2

[xianbaoqian]

Context huggingface/hub-docs#552 (comment)

It's called k2_sherpa in the inference API but in the repos, people just use k2 so let's make it consistent so that the export script can work.

[osanseviero]

LGTM but it might require some changes in api-inference-community.

@Narsil I don't see k2 in places such as https://github.com/huggingface/api-inference/blob/main/master/app/types.py#L21, but the API works. How come? :D

[osanseviero]

Would you mind adding a second workflow as this one? https://github.com/huggingface/api-inference-community/blob/main/.github/workflows/python-api-flair-cd.yml

That will enabled continuous deployment

[xianbaoqian]

LGTM but it might require some changes in api-inference-community.

@Narsil I don't see k2 in places such as https://github.com/huggingface/api-inference/blob/main/master/app/types.py#L21, but the API works. How come? :D

Yeah. I'm curious about this too. I had a quick code search on the repo but didn't find many things. This repo has k2 tag but the inference API is working.

https://huggingface.co/csukuangfj/icefall-asr-librispeech-pruned-transducer-stateless3-2022-05-13

[Narsil]

It's defined right there: https://github.com/huggingface/api-inference/blob/main/master-rs/src/data/hf_types.rs#L237

[Narsil]

The rename should work.

[Narsil]

The rename should work.

Unfortunately the build is now broken, I don't know what maintainers have done.

[Narsil]

LGTM

[osanseviero]

Out of curiosity, why is it in https://github.com/huggingface/api-inference/blob/main/master-rs/src/data/hf_types.rs#L237 but not in https://github.com/huggingface/api-inference/blob/main/master/app/types.py#L21 ?

[xianbaoqian]

Thanks for fixing the bug @Narsil !