search

huggingface / api-inference-community

Apache License 2.0
161 stars 61 forks source link

Remove Keras from InferenceAPI (community) #434

Closed Wauplin closed 2 months ago

Wauplin commented 2 months ago

Related to https://github.com/keras-team/keras/pull/19854, https://github.com/huggingface/huggingface.js/pull/774, https://github.com/huggingface/hub-docs/pull/1320 and, https://github.com/huggingface/moon-landing/pull/10501 (private).

See https://github.com/huggingface/api-inference/pull/1956 for complete drop in private repo.

As discussed in https://github.com/huggingface/moon-landing/pull/10501#issuecomment-2196278636 (private), it's better to drop Keras models in Inference API, at least for now. This choice was motivated by the fact that:

  1. only image-classification is supported
  2. meaning only 223 models totaling 1300 downloads over the last 30 days are affected
  3. when https://github.com/huggingface/moon-landing/pull/10501 (private) is merged, legacy keras models will be tagged as tf-keras, thus breaking the current Inference API

Given its low usage and scope, let's just drop the support. If keras becomes a major framework on the platform (which we hope :crossed_fingers: ), then we can reassess.